RFS: png2html (updated package)

Dear mentors,

I am looking for a sponsor for the new version 1.1-5 of my
package "png2html"
(This package was previously orphaned). I would like to
thank those who have
taken the time to look at this package and make suggestions
for improvements,
and I hope it is now ready for sponsorship.

It builds these binary packages:
png2html   - transforms a PNG image to a web page

The package is lintian clean.

The upload would fix these bugs: 338971, 387477, 388602,
343915, 335659

The package can be found on mentors.debian.net:
- URL: http://mentors.debian.net/debian/pool/main/p/png2html
- Source repository: deb-src http://mentors.debia
n.net/debian unstable main
contrib non-free
- dget http://mentors.debian.net/debian/pool/ma
in/p/png2html/png2html_1.1-5.dsc

I would be glad if someone uploaded this package for me.

Kind regards
Jack Grahams

Hello there,

On Tue, Sep 26, 2006 at 04:29:16PM +0100, Jack Grahams
wrote:
> I am looking for a sponsor for the new version 1.1-5 of
my package "png2html"
> (This package was previously orphaned). I would like to
thank those who have
> taken the time to look at this package and make
suggestions for improvements,
> and I hope it is now ready for sponsorship.

Some further notes on your debian/rules, partially being
nitpicky:

The CFLAGS are not passed to make, so setting -g effectively
yields
no result as upstream uses his own set of flags. Beware, he
mixes
flags and libraries in his CFLAGS!

DH_COMPAT is supposed to be used to temporarily override
your normal
settings as specified in debian/compat, so you might want to
transfer
setting the compatiblity level into this file.

DH_VERBOSE=1 might be considered too noisy for an upload
into the
Debian archives.

Other than that the packaging looks clean to me, and I'm
interested in
sponsoring. Please check these notes and approach me again
once the
package is ready to be uploaded. Provided, of course, nobody
beats me
to it. 

Cheers,
Flo

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Flo,

> The CFLAGS are not passed to make, so setting -g
effectively yields
> no result as upstream uses his own set of flags.

Good point, I hadn't noticed that - I thought I'd got rid of
all the
pointless parts of debian/rules...

> DH_COMPAT is supposed to be used to temporarily
override your normal
> settings as specified in debian/compat

Thanks for the heads-up - I didn't know that - I obviously
didn't look
hard enough in the Policy or Developers' manual!

> DH_VERBOSE=1 might be considered too noisy for an
upload into the
> Debian archives.

True!

> Other than that the packaging looks clean to me, and
I'm interested in
> sponsoring. Please check these notes and approach me
again once the
> package is ready to be uploaded.

Okay, I've uploaded a copy with your fixes to
mentors.debian.net. I'd
very much appreciate it if you would sponsor it  (provided
you're
happy with the changes of course...)

Thanks,
Jack

http://mentors.debian.net/debian/pool/main/p/png2html







-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQFFGXSU7l4lrvUNGf4RAg75AJ4ifZF3AlsBt81GJ8L01T6AzkLCwACd
Gz+k
Ei9qqSadCQ0P6pWKua9IfLk=
=BUYG
-----END PGP SIGNATURE-----


-- 
To UNSUBSCRIBE, email to debian-mentors-REQUESTlists.debian.org
with a subject of "unsubscribe". Trouble? Contact
listmasterlists.debian.org

On Tue, Sep 26, 2006 at 07:42:56PM +0100, Jack Grahams
wrote:
> > The CFLAGS are not passed to make, so setting -g
effectively yields
> > no result as upstream uses his own set of flags.
> 
> Good point, I hadn't noticed that - I thought I'd got
rid of all the
> pointless parts of debian/rules...

Umm, that's not quite what I meant. 

"any binaries created should include debugging
information", please
see <http://www.debian.org/doc/debian-policy/ch-files#s10.
1>. Failing
to comply with this part of Policy will warrant a bugreport,
so what I
meant was: please take care that the CFLAGS from your
previous
debian/rules get passed on correctly.

> > DH_COMPAT is supposed to be used to temporarily
override your normal
> > settings as specified in debian/compat
> 
> Thanks for the heads-up - I didn't know that - I
obviously didn't look
> hard enough in the Policy or Developers' manual!

Both texts better shouldn't go into too much detail about
how certain
helper tools are supposed to work. man debhelper reveals
that part. 

Cheers,
Flo


PS: I'm on the list, there is no need to CC me, thanks!