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

Re: RFS: fizsh



Hi Michael,

I fixed the two remaining issues (added a patch for the man page, and
used dput to upload the original version of fizsh_1.0.2.orig.tar.gz).

The amended package and the original version have been uploaded as:

* http://mentors.debian.net/debian/pool/main/f/fizsh/fizsh_1.0.2.orig.tar.gz
* http://mentors.debian.net/debian/pool/main/f/fizsh/fizsh_1.0.2-1.debian.tar.gz

The package can be downloaded with the command: "dget -x
http://mentors.debian.net/debian/pool/main/f/fizsh/fizsh_1.0.2-1.dsc";

I hope the package is ready for uploading to Debian now.

Thank you very much for all your help!

Guido


On Sun, Sep 4, 2011 at 7:36 PM, Guido van Steen
<vansteen@users.sourceforge.net> wrote:
> Hi Michael,
>
> Thanks for looking at my package again.
>
>> - why not fix the hyphen-used-as-minus-sign via a patch instead of overriding
>>  the warning? Overriding a correct warning for the mere reason of silencing
>>  lintian seems to be of no use.
>
> I first tried to create a patch, but I could not add it in a correct way.
>
> Then I thought that arguments to commands (such as "-a" in "ls -a")
> start with hyphens, and not with minus signs, so that lintian was
> actually wrong. I may be wrong about this though. Anyway for this
> reason I added a "lintian-override" file.
>
> I will add working patch in the next upload.
>
>> - The tar.gz file differs in the same way it did before. Why don't you use
>>  exactly the file that can be downloaded from sf.net?
>
> I am very sorry. It is only now that I see that this is due to a bug
> in a script I used. This bug will not be present in the next upload.
>
> Best wishes,
>
> Guido
>
>
> On Sun, Sep 4, 2011 at 5:29 PM, Michael Tautschnig <mt@debian.org> wrote:
>> Hi Guido,
>>
>>> Dear Michael, and other mentors,
>>>
>>> Thank you very much for reviewing my package!
>>>
>>> I corrected the errors you spotted. I also added a "lintian-override"
>>> file to suppress the lintian "hyphen-used-as-minus-sign" tag. (This
>>> "override-file" will not be needed anymore when Upstream releases a
>>> newer version of fizsh.)
>>>
>> [...]
>>
>> I understand that you fixed some of these errors, but
>>
>> - why not fix the hyphen-used-as-minus-sign via a patch instead of overriding
>>  the warning? Overriding a correct warning for the mere reason of silencing
>>  lintian seems to be of no use.
>>
>> Best,
>> Michael
>>
>>
>


Reply to: