Re: update - ccdoc 08.41 is ready for experimentation

[Reini Urban <rurban at x-ray dot at>]

Reini Urban schrieb:
> You need a subject like: "[ITP] ccdoc 08.41"

Why this funny version number?
It should be called "ccdoc-0.8.41" imho.
Or if MAJOR is 8 leave the 0 away => ccdoc-8.41
This is irritating.

BTW: This is the longest -h message I've ever seen.
Was the intent to make "ccdoc -h" be longer then "man ccdoc" :)
Just joking.

> And then a sentence in which you state that you want to maintain this 
> cygwin package. The reviewers will point to problems in your packaging 
> or licensing.
>
> "upload" is reserved for packages which already got their 3 positive 
> reviews and one GTG (good to go).

err, "update", you know what I mean.


ok, to the review:
binary package looks fine.

src package has /usr/src/ccdoc-08.41-1.patch hardcoded,
which may not be! even if it's empty.
The rest is simple, but okay.

Please just remove the patch file from this path.
You might want to add this empty ccdoc-08.41-1.patch to topdir.
but then you have to call tar cfj differently.

With these fixes +1 and GTG.

At least I have now a full cygwin class documentation:
$ cd /usr/src
$ ccdoc -db /tmp/ccdoc.db -pkg Cygwin src/winsup/cygwin/*.h
$ ccdoc -db /tmp/ccdoc.db -index
$ mkdir -p cygwin-ccdocs/images  # required
$ ccdoc -db /tmp/ccdoc.db -html cygwin-ccdocs/ -imgurl images/ -root 
winsup -rooturl winsup.html