[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#362584: xindy debian package



Jörg Sommer <joerg@alea.gnuu.de> wrote:

> Hallo Norbert,
>
> Norbert Preining schrieb am Mon 05. Feb, 10:41 (+0100):
>> On Mon, 05 Feb 2007, Frank Küster wrote:
>> > - wouldn't it better to patch the Makefile to disable maintainer mode,
>> >   instead of setting all those variables?  And wouldn't it make sense to
>> >   patch the Makefile sources, too, to allow easy submission of the
>> >   patches to upstream?
>> > 
>> > - Isn't it possible (and if yes, a cleaner approach) to pass --docdir 
>> >   already to the configure call, not make install?
>> 
>> Other suggestions by me:
>> - please repackage the .orig.tar.gz: You include the whole clisp code
>>   for nothing, which makes up 90% of the unpacked sources, but AFAIS
>>   it is not used for anything.
>
> I would prefer to leave the source as it is to keep transparency.

You're the maintainer, and it's up to you to decide this.  

>> Also the call to dh_perl seems to be useless (but this I didn't check
>> in detail)
>
> texindy and xindy are perl files. dh_perl is needed to get the right
> dependency on perl. Or I am wrong?

No, I don't think so - but there's no ${perl:Depends} in the control
file, so dh_perl's result is just discarded.  At least that's how it
looks like, I didn't try it, maybe I'm missing something.

Regards, Frank
-- 
Dr. Frank Küster
Single Molecule Spectroscopy, Protein Folding @ Inst. f. Biochemie, Univ. Zürich
Debian Developer (teTeX/TeXLive)



Reply to: