[Pkg-awstats-devel] I'd better leave
Jonas Smedegaard
dr at jones.dk
Wed Mar 16 21:04:36 UTC 2011
On Wed, Mar 16, 2011 at 11:08:50PM +0300, Sergey B Kirpichev wrote:
>On Wed, Mar 16, 2011 at 10:43 PM, Jonas Smedegaard <dr at jones.dk> wrote:
>> But no, it is not the _reason_ for it: I felt frustrated even looking
>> at those scripts. Because they are written in another style than I
>> would do - better in some ways, but different. And their purpose is
>> different from how I would like them to provide an interface for our
>> users.
>
>And... how exactly you would like to provide one? Clearly, besides of
>tiny style fixes, you have a lot of vision how to do this all in the
>right way (TM).
>Share this knowledge, even random thoughts and some draft ideas!
I would...
* avoid using a tempfile
* avoid changing directory
* avoid using ls
* avoid hardcoded paths for utilities in default $PATH
* not require certain order of processing
* use $() instead of backticks
* quote variables more aggressively
None of that is about right or wrong, but about style. As I already
emphasized in my previous mail.
More fundamentally, however, I dislike that update.sh script being
hooked onto to cronjob in the first place, because it makes us then
promise to maintain that way of organizing multiple hosts, and I don't
find it a flexible enough approach. Having introduced it makes it
difficult to later change.
Again, it is not about right or wrong, but about style.
>> If you ever want me co-maintaining some other packages then please
>> try me - I enjoyed working with you and would love to try again:
>> Leaving now isn't personal at all! :-)
>
>I've no such a clearly complex stuff to be needed in co-maintaince,
>exept for awstats, but you could look on the monit package, for
>example.
If not really needed then I'd rather not: I am involved in 200+ packages
currently so not _desparate_ to get involved in more. :-)
- Jonas
--
* Jonas Smedegaard - idealist & Internet-arkitekt
* Tlf.: +45 40843136 Website: http://dr.jones.dk/
[x] quote me freely [ ] ask before reusing [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-awstats-devel/attachments/20110316/9195c46d/attachment.pgp>
More information about the Pkg-awstats-devel
mailing list