Hi

Dne Wed, 14 Nov 2012 18:50:45 +0100
intrigeri <intrig...@debian.org> napsal(a):

> Hi,
> 
> Michal Čihař wrote (14 Nov 2012 07:53:12 GMT) :
> > Okay, it looks like I need to use same (ugly) hack as xdebug is
> > using, will do another upload.
> 
> I just tried the same upgrade path again (s,-3,-4,), and as a result
> my modification change is still in xcache.ini, the (correct) absolute
> extension path is in there too.
> 
> There's no .dpkg-* files around. I guess at least if there was new
> default settings brought by the new conffile, .dpkg-dist should be
> here, but I've no idea if it is the case. Is it?

It should be. However it won't be there for upgrades from versions
which did already move config file without using dpkg helper.
Unfortunately the helper is not clever enough to handle breakage I've
introduced.

> About the extension path: after testing, I first thought "oh this is
> still broken", because "Do not include full path to extension in
> xcache.so." in the 2.0.0-3 debian/changelog made me think the desired
> state was to *not* have the path at all in the ini file, so I thought
> that would be done in the new postinst thing. But it looks like it
> works fine in practice! I am confident you have tested the stable ->
> 2.0.0-4 upgrade path, so that one should be good too. So far, so good.
> 
> The whole thing is still unclear and prone to misunderstanding,
> when reading debian/changelog. How about:
> 
>   * Either the ugly postinst hack *removes* the absolute extension
>     path from the ini file, and the ini file does not include it to
>     start with -- if this is supported, which is suggested by -3 -- is
>     it? If this is possible, it would avoid having to deal with that
>     for Jessie again...
> 
>   * Or a new debian/changelog entry should clarify that eventually,
>     we're back to including the full path.

The absolute path unfortunately needs to be there - that's difference
between zend_extension and extension. Fortunately the path removal was
done in file which was not used at the end. Anyway as the package is
now unblocked I'd like to avoid yet another upload just for
changelog clarification.


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

Attachment: signature.asc
Description: PGP signature

Reply via email to