Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-24 Thread Bernhard R. Link
* Andreas Hoenen [EMAIL PROTECTED] [080123 22:19]:
 causes patch-stamp to be executed as first action before the build-stamp
 commands get executed.  Symmetrical to this the construct
 
 clean: clean1 unpatch
 clean1:
 
 causes unpatch to be executed as last action after all clean1 commands.

AFAIK that might happen, but only by coincidence (i.e. specific
implementation of make and no -j arguments). To make sure unpatch
is called after clean1, make unpatch depend on clean1.

Hochachtungsvoll,
Bernhard R. Link


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-24 Thread Andreas Hoenen
Michal Čihař [EMAIL PROTECTED] wrote:

snip/
 Uploaded, thanks for your effort. Next time you want to upload it,
 please ask directly me with [sponsoring] in subject.
 
 -- 
   Michal Čihař | http://cihar.com | http://blog.cihar.com

Thanks for your upload.  I will do as you suggested.

Regards, Andreas
-- 
Andreas Hoenen [EMAIL PROTECTED]

GPG: 1024D/B888D2CE
 A4A6 E8B5 593A E89B 496B
 82F0 728D 8B7E B888 D2CE


pgpSzCOO8uwnj.pgp
Description: PGP signature


Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-23 Thread Andreas Hoenen
Michal Čihař [EMAIL PROTECTED] wrote:

Hello,

 Hi

 On Tue, 22 Jan 2008 21:12:02 +0100
 Andreas Hoenen [EMAIL PROTECTED] wrote:

  Michal Čihař [EMAIL PROTECTED] wrote:
   Any reason why not to use /usr/share/dpatch/dpatch.make and including
   dpatch rules direcly?
 
  Both ways are described as possible in the dpatch man page, but yours
  looks more elegant.  Adapted.

 Where does clean1 target come from now? I don't see any reason for it.

Just taken from the example in the manual page DPATCH.MAKE(7).  I think
the reason for this indirect approach is as follows:

build-stamp: patch-stamp

causes patch-stamp to be executed as first action before the build-stamp
commands get executed.  Symmetrical to this the construct

clean: clean1 unpatch
clean1:

causes unpatch to be executed as last action after all clean1 commands.
Whereas

clean: unpatch

means unpatch being executed before the clean commands.  For dblatex it
seems to be a matter of taste, as the stack approach is not needed.
Thus adapted.


   I think it is cleaner not to have things like if [ -f
   debian/lintian.overrides ] in debian/rules. You know whether there is
   lintian override when you create package.
 
  My argumentation is as follows: For some releases I need overrides, for
  others not.  I prefer to change only one location (overrides file
  itself) than additionally the rules file.  But this seems to be
  arguable.  Thus changed.

 It's better to fix reason why you need override than adding one on so,e
 releases.

In general you are right, however I added the override for a false
positive from lintian [1].  (Although there Russ Allberry seems to
advise not to override lintian bugs.)  Of course I prefer obeying
packaging standards to overriding warnings about real violations.  ;-)


 And one more note - watch file could be simplified to this and use
 uscan support for sourceforge:
 http://sf.net/dblatex/dblatex-(.*)\.tar\.bz2

 --
   Michal Čihař | http://cihar.com | http://blog.cihar.com

Adapted.  Thus the next release candidate is:

- URL: http://mentors.debian.net/debian/pool/main/d/dblatex
- Source repository: deb-src http://mentors.debian.net/debian unstable main 
contrib non-free
- dget http://mentors.debian.net/debian/pool/main/d/dblatex/dblatex_0.2.8-5.dsc

[1] http://lists.debian.org/debian-mentors/2007/12/msg00264.html

Regards, Andreas
-- 
Andreas Hoenen [EMAIL PROTECTED]

GPG: 1024D/B888D2CE
 A4A6 E8B5 593A E89B 496B
 82F0 728D 8B7E B888 D2CE


pgpXNMBkKNx9t.pgp
Description: PGP signature


Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-23 Thread Michal Čihař
Hi

On Wed, 23 Jan 2008 21:58:27 +0100
Andreas Hoenen [EMAIL PROTECTED] wrote:

 Adapted.  Thus the next release candidate is:
 
 - URL: http://mentors.debian.net/debian/pool/main/d/dblatex
 - Source repository: deb-src http://mentors.debian.net/debian unstable main 
 contrib non-free
 - dget 
 http://mentors.debian.net/debian/pool/main/d/dblatex/dblatex_0.2.8-5.dsc

Uploaded, thanks for your effort. Next time you want to upload it,
please ask directly me with [sponsoring] in subject.

-- 
Michal Čihař | http://cihar.com | http://blog.cihar.com


signature.asc
Description: PGP signature


Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-22 Thread Andreas Hoenen
Michal Čihař [EMAIL PROTECTED] wrote:

Thanks for your review.

 Hi

 On Sat, 19 Jan 2008 11:04:52 +0100
 Andreas Hoenen [EMAIL PROTECTED] wrote:

  [3] - URL: http://mentors.debian.net/debian/pool/main/d/dblatex
  - Source repository: deb-src http://mentors.debian.net/debian unstable 
  main contrib non-free
  - dget 
  http://mentors.debian.net/debian/pool/main/d/dblatex/dblatex_0.2.8-3.dsc

 Just few comments (not necessary errors):

 Any reason why not to use /usr/share/dpatch/dpatch.make and including
 dpatch rules direcly?

Both ways are described as possible in the dpatch man page, but yours
looks more elegant.  Adapted.


 I think it is cleaner not to have things like if [ -f
 debian/lintian.overrides ] in debian/rules. You know whether there is
 lintian override when you create package.

My argumentation is as follows: For some releases I need overrides, for
others not.  I prefer to change only one location (overrides file
itself) than additionally the rules file.  But this seems to be
arguable.  Thus changed.


 $ lintian -I dblatex_0.2.8-3_i386.changes
 I: dblatex source: build-depends-without-arch-dep python-support

Adapted.  I really should use the -I switch with lintian.  :-(


 How about cleaning up dependencies on ghostscript? gs-gpl is a
 transitional package and gs-pdfencrypt does not seem to exist.

(gs-pdfencrypt is a virtual package.)  Cleaned up.


 Otherwise it looks in quite good shape and I think I can upload it.

 --
   Michal Čihař | http://cihar.com | http://blog.cihar.com

That would be great.  I have uploaded adapted version 0.2.8-4:

- URL: http://mentors.debian.net/debian/pool/main/d/dblatex
- Source repository: deb-src http://mentors.debian.net/debian unstable main 
contrib non-free
- dget http://mentors.debian.net/debian/pool/main/d/dblatex/dblatex_0.2.8-4.dsc

Thanks, Andreas
--
Andreas Hoenen [EMAIL PROTECTED]

GPG: 1024D/B888D2CE
 A4A6 E8B5 593A E89B 496B
 82F0 728D 8B7E B888 D2CE


pgpRn2ktmJhlX.pgp
Description: PGP signature


Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-22 Thread Michal Čihař
Hi

On Tue, 22 Jan 2008 21:12:02 +0100
Andreas Hoenen [EMAIL PROTECTED] wrote:

 Michal Čihař [EMAIL PROTECTED] wrote:
  Any reason why not to use /usr/share/dpatch/dpatch.make and including
  dpatch rules direcly?
 
 Both ways are described as possible in the dpatch man page, but yours
 looks more elegant.  Adapted.

Where does clean1 target come from now? I don't see any reason for it.

  I think it is cleaner not to have things like if [ -f
  debian/lintian.overrides ] in debian/rules. You know whether there is
  lintian override when you create package.
 
 My argumentation is as follows: For some releases I need overrides, for
 others not.  I prefer to change only one location (overrides file
 itself) than additionally the rules file.  But this seems to be
 arguable.  Thus changed.

It's better to fix reason why you need override than adding one on so,e
releases.

And one more note - watch file could be simplified to this and use
uscan support for sourceforge:
http://sf.net/dblatex/dblatex-(.*)\.tar\.bz2

-- 
Michal Čihař | http://cihar.com | http://blog.cihar.com


signature.asc
Description: PGP signature


Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-20 Thread Kapil Hari Paranjape
Hello,

On Sat, 19 Jan 2008, Andreas Hoenen wrote:
 as my default sponsor recently has not found the time for sponsoring,
 and as he agrees I should better search another sponsor - well, here I
 am :-)

and:
 To finalize my advertisement: the dblatex package is not very
 complicated (architecture all), with a frequency of at most one release
 per month.  However, IMHO it's a useful and important package, being the
 sole DocBook-PDF tool in Debian main.  And although I'm making
 packaging mistakes, usually I learn and avoid repeating them.  My GPG
 key is signed by one DD, thus I'm basically connected to the web of
 trust.
 
 I hope to have given a picture clear enough to base a sponsoring
 decision upon.  Thus, if someone is interested, I really would be glad.

Since this package is based on python, it would probably be a good
idea to ask on the debian-python mailing list.

Regards,

Kapil.
--


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Re: RFS: dblatex: long term / current release 0.2.8-3

2008-01-20 Thread Michal Čihař
Hi

On Sat, 19 Jan 2008 11:04:52 +0100
Andreas Hoenen [EMAIL PROTECTED] wrote:

 [3] - URL: http://mentors.debian.net/debian/pool/main/d/dblatex
 - Source repository: deb-src http://mentors.debian.net/debian unstable 
 main contrib non-free
 - dget 
 http://mentors.debian.net/debian/pool/main/d/dblatex/dblatex_0.2.8-3.dsc

Just few comments (not necessary errors): 

Any reason why not to use /usr/share/dpatch/dpatch.make and including
dpatch rules direcly?

I think it is cleaner not to have things like if [ -f
debian/lintian.overrides ] in debian/rules. You know whether there is
lintian override when you create package.

$ lintian -I dblatex_0.2.8-3_i386.changes 
I: dblatex source: build-depends-without-arch-dep python-support

How about cleaning up dependencies on ghostscript? gs-gpl is a
transitional package and gs-pdfencrypt does not seem to exist.

Otherwise it looks in quite good shape and I think I can upload it.

-- 
Michal Čihař | http://cihar.com | http://blog.cihar.com


signature.asc
Description: PGP signature


RFS: dblatex: long term / current release 0.2.8-3

2008-01-19 Thread Andreas Hoenen
Dear mentors,

as my default sponsor recently has not found the time for sponsoring,
and as he agrees I should better search another sponsor - well, here I
am :-)

Currently I'm the maintainer of dblatex [1] and co-maintainer of
apt-show-versions [2].  Thus my involvement is limited, but I set a high
value on doing my job properly and long-ranging - thus I only take over
a package if I'm really and durable interested.  a-s-v is sponsored by
the package owner (as a native package it's his child), thus
sponsoring is needed for dblatex.  Current release awaiting upload is
0.2.8-3 on mentors.debian.net [3].

At the moment I have no plans to become a DD, being confident with my
current level of involvement.  This might change sometime, but I won't
count on it.  On the other hand, I'm definitely interested in becoming a
DM - it looks convenient for all parties and expresses trust in my
packaging.  But that is second-rank, first priority is to get dblatex
releases published in a timely manner, which is my reponsibility to its
users.

To finalize my advertisement: the dblatex package is not very
complicated (architecture all), with a frequency of at most one release
per month.  However, IMHO it's a useful and important package, being the
sole DocBook-PDF tool in Debian main.  And although I'm making
packaging mistakes, usually I learn and avoid repeating them.  My GPG
key is signed by one DD, thus I'm basically connected to the web of
trust.

I hope to have given a picture clear enough to base a sponsoring
decision upon.  Thus, if someone is interested, I really would be glad.

[1] http://packages.debian.org/sid/dblatex
Produces DVI, PostScript, PDF documents from DocBook sources
[2] http://packages.debian.org/sid/apt-show-versions
lists available package versions with distribution
[3] - URL: http://mentors.debian.net/debian/pool/main/d/dblatex
- Source repository: deb-src http://mentors.debian.net/debian unstable main 
contrib non-free
- dget 
http://mentors.debian.net/debian/pool/main/d/dblatex/dblatex_0.2.8-3.dsc

Regards, Andreas
--
Andreas Hoenen [EMAIL PROTECTED]

GPG: 1024D/B888D2CE
 A4A6 E8B5 593A E89B 496B
 82F0 728D 8B7E B888 D2CE


pgpvoEsG1KNc9.pgp
Description: PGP signature