[Pkg-awstats-devel] I'd better leave

[Jonas Smedegaard]

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>