Hi Jakub

2015-12-27 6:41 GMT-05:00 Jakub Wilk <jw...@debian.org>:
> * Daniel Echeverry <epsilo...@gmail.com>, 2015-12-26, 23:56:
>>
>> http://mentors.debian.net/debian/pool/main/s/sxiv/sxiv_1.3.2-1.dsc
>
>
> Typo in the patch name: behaivor -> behavior

Fixed!

>
> You might want to add the Bugs-Debian header to the patch header.
>

Fixed!

> exec/key-handler uses temporary files insecurely.
>

Could you give me some info about  this? I am confused, please point
me out to some useful url, thanks

> Lintian says:
> X: sxiv: binary-file-built-without-LFS-support usr/bin/sxiv
>
> Now, I doubt sxiv could do anything sensible with >2GB images on a 32-bit
> system anyway. But it turns out that you can be bitten by lack on LFS
> support even when all your files are small:
> https://lists.debian.org/20150712173723.ga29...@gaara.hadrons.org
>
> sxiv calls stat() in multiple places, so I think you should enable LFS.

Fixed!

>
> --
> Jakub Wilk
>

Thank for you review!

Regards

-- 
Daniel Echeverry
http://wiki.debian.org/DanielEcheverry
Linux user: #477840
Debian user
Software libre

Reply via email to