[aur-general] Proofreading request

Evan Teitelman teitelmanevan at gmail.com
Tue Aug 27 13:19:30 EDT 2013

On Mon, Aug 26, 2013 at 11:17:16PM +0200, Cl?ment Junca wrote:
> Could someone have a look at the attached package, please ? It's my first
> AUR package written from scratch.

A few additional notes:
 - Your patch is small. You can use `sed` instead.
       sed -i 's|env python|env python2|' feather
       sed -i 's|binpath: /usr/local/bin/|binpath: /usr/bin/|' feather.yaml.dist
 - Here is a fixed and cleaned version of your `config.install` file:
       # You should probably run this on install, not on upgrade.
       # If you want to run this on both install and upgrade, you can do this:
       #   post_install() {
       #      post_upgrade
       #   }
       post_install() {
         # Install file output should be dense.
         echo 'Feather is designed to be run from cron like this:'
         # The quotation marks are necessary here. Without them, the
shell will expand the asterisks.
         echo '    */5 * * * * /usr/bin/feather /etc/feather.yaml'
         # 'shall' gives the impression that the file will be updated
         echo 'Configuration file /etc/feather.yaml must be updated
before running.'
- It is common to name install files after the packages they belong
to. You should consider naming your install file `feather.install` or

More information about the aur-general mailing list