r8085 and r8104

I've just added a "concept" +1 to this item in STATUS:

   * r8055, r8085, r8104, r8105
     RPM Documentation build patch and added svnserve.conf and snvserve man
        pages. Also require correct apr/apr-util, and apache RPM versions.
     Justification: Packaging fixes and additions.
     Votes:
      +1 (concept): gstein, kfogel

The middle two revisions there are really the same patch, applied to
both rpms/redhat-7.x/ and rpms/redhat-8+/.

David, I can change my vote to a real +1 if you can confirm to me that
"%{__perl}" lines in the patch work exactly as intended. That's the
one part I didn't feel competent to review, but I'm happy to go out on
a limb for you if you've tested this :-). Also, you might want to
vote on this item yourself!

One thing:

In those two revisions, part of the patch looks like this:

  +# Start patch by Ben Reser <ben@reser.org> to get documentation to build.
  +%{__perl} -pi -e [...]
  +%{__perl} -pi -e [...]
  +# End patch by Ben Reser <ben@reser.org> to get documentation to build.

For the future: when applying a patch, you can take out the markers
left by the contributor, and (if appropriate) replace them with a more
timeless comment, for example:

  +# Build documentation.
  +%{__perl} -pi -e [...]
  +%{__perl} -pi -e [...]

If we always left comments like this in the code

   ### JRANDOM: here's the start of my mods
   [...]
   ### JRANDOM: here's the end of my mods

... well, let's just say we wouldn't need version control :-).

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jan 7 22:06:01 2004