Beefy Boxes and Bandwidth Generously Provided by pair Networks
Think about Loose Coupling

Re^2: Feedback on my second script

by Jackiejarvis (Novice)
on May 15, 2013 at 10:11 UTC ( #1033652=note: print w/replies, xml ) Need Help??

in reply to Re: Feedback on my second script
in thread Feedback on my second script

Thanks for the feedback. I have inserted the things you suggest. About the running rsync. In the future I would send a mail to the admin if there is an error with the error message included. How could i do something like this is it possible if i use system or not?


Replies are listed 'Best First'.
Re^3: Feedback on my second script
by Corion (Pope) on May 15, 2013 at 10:19 UTC

    Yes, if you want to extend the program later to send the output via mail (but only in the case of error), your current approach makes sense.

Re: Feedback on my second script
by jonadab (Parson) on May 15, 2013 at 10:47 UTC
    I would send a mail... How could i do something like this

    I've traditionally used Mail::Sendmail for that; although, Mail::Send may also be worth considering.

    Note too that it's generally unsafe to allow the user to specify a destination address for mail (unless you have carefully authenticated the user). In simple cases like this, it's often best to hardcode the destination address. In more advanced cases, possible destination addresses can be stored in a configuration file or database that only system administrators can edit, and the user can be allowed to select from among them.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1033652]
[Corion]: If all else fails, Spreadsheet::XLSX should be "installable" by manually copying the files, provided that the prerequisites are met
[thezip]: It fails in the "formatted 2-digit numeric" tests
[thezip]: Okay Corion, that's what I wanted to hear. Thanks!
[Corion]: Oh, a test failure... Just force-install it then? --force and potentially --notest to skip the tests alltogether ?
[ambrus]: thezip: is Activestate Perl usable for your scenario instead?
[MidLifeXis]: Yeah, --notest is good, not sure I would do --force.
[thezip]: It fails 2/10 tests in that group. What are the ramifications if I force it? What should I look for for?
[thezip]: I've migrated from ActiveState to Strawberry. No going back...
[thezip]: I'll try the --notest arg first when I get back from meetings (after lunch).
[thezip]: Thanks guys for your comments! :-)

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (13)
As of 2017-03-23 17:18 GMT
Find Nodes?
    Voting Booth?
    Should Pluto Get Its Planethood Back?

    Results (290 votes). Check out past polls.