Bug#580880: pgfouine: changing back from O to ITA
Hi Christoph Thanks for looking at the package. On Mon, Apr 04, 2011 at 05:52:50PM +0200, Christoph Berg wrote: I've just had a look at your pgfouine package. Do you have any idea what debian/patches/10-include-path is good for, except for introducing gapping security holes? IMHO this one should be removed. +ini_set( 'include_path', '.:/usr/share/pgfouine/'); I agree. Remove the '.' from the ini_set() Re: Luis Uribe 2011-02-27 20110227161853.GA26585@volatile I just noticed a [3]new bug on the package that i'm investigating [3] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=612779 Did you resolve that in the meantime? No, i can't reproduce it with some of my logs so i need to look deeper. Otherwise the package looked good (though I wasn't really finished with looking yet). I just upload a new version to mentors: fix the patch and bump the Standards-Version http://mentors.debian.net/debian/pool/main/p/pgfouine/pgfouine_1.2-3.dsc Regards, -- Luis http://eviled.org signature.asc Description: Digital signature
Bug#580880: pgfouine: changing back from O to ITA
Hi Luis, I've just had a look at your pgfouine package. Do you have any idea what debian/patches/10-include-path is good for, except for introducing gapping security holes? IMHO this one should be removed. +ini_set( 'include_path', '.:/usr/share/pgfouine/'); Re: Luis Uribe 2011-02-27 20110227161853.GA26585@volatile I just noticed a [3]new bug on the package that i'm investigating [3] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=612779 Did you resolve that in the meantime? Otherwise the package looked good (though I wasn't really finished with looking yet). Christoph -- c...@df7cb.de | http://www.df7cb.de/ signature.asc Description: Digital signature
Bug#580880: pgfouine: changing back from O to ITA
retitle 580880 ITA: pgfouine -- PostgreSQL log analyzer owner 580880 ! thanks There is a package already in [1]mentors.d.n [1] http://mentors.debian.net/debian/pool/main/p/pgfouine/pgfouine_1.2-2.dsc And a [2]thread with link to other threads in debian-mentors@l.d.o [2] http://lists.debian.org/debian-mentors/2011/01/msg00354.html I just noticed a [3]new bug on the package that i'm investigating [3] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=612779 Cheers, -- Luis http://eviled.org signature.asc Description: Digital signature
