On Wed, Oct 11, 2006, Ralf S. Engelschall wrote:
> OpenPKG CVS Repository
> http://cvs.openpkg.org/
> ____________________________________________________________________________
>
> Server: cvs.openpkg.org Name: Ralf S. Engelschall
> Root: /v/openpkg/cvs Email: [EMAIL PROTECTED]
Mr Cosmetics again ... Hi, haven't seen you for a while ;-)
> Module: openpkg-src Date: 11-Oct-2006 22:53:42
> Branch: HEAD Handle: 2006101121534200
>
> Modified files:
> openpkg-src/varnish varnish.patch varnish.spec
>
> Log:
> - bugfix: exit(1) in case the PID file is not writable or
> a segfault would happen by the following fprintf/fclose.
Well, I riped out the hunk from similar piece of code (by you :) in
qpopper.patch. Should it be fixed there as well?
> - cleanup: do not keep the C library files in the package as they
> are both useless without the corresponding C header files and even with
> them they are not really useful as varnish is a self-contained
> application.
I'm currently in the process of evaluating Varnish (this is pretty cool
stuff :). Documentation is still somewhat incomplete at these early
stages of Varnish so there are many black holes to discover. As there
is an API for third party plugins I wasn't sure about the libs and kept
them. If you have checked that they are obsolete (maybe I just missed
the add the header files manually), it's absolutly okay to dump them.
> - consistency: use "switch" instead of "osdn" for SourceForge URLs
> to be in sync with all(!) other packages (although
> we know that the host "osdn" is available again).
osdn still isn't available all the times. Why don't we define a macro
%{l_sfhost} for that purpose instead of switching all over again in
many, many packages? This shouldn't be general rule of thumb but in case
of SourceForge this would be really helpful.
> - cosmetics: use our usual 'all-caps' style in Summary header
Uh, yeah but I don't like it anyway :-)
> - cosmetics: use 0-prefix patch to avoid hard-coded versions
> in patch files when easily possible
BTW can we rid of those 0-prefix patches you'll find in all that perl-*
packages. Why not preprocessing the patches doing a %{V_foo_bar} subst?
We have to touch perl-* much more often that we will in case of Varnish.
-cs
______________________________________________________________________
The OpenPKG Project www.openpkg.org
Developer Communication List [email protected]
