On 08/15/2010 01:40 PM, Raphael Hertzog wrote:
> reassign 593043 libconfig-simple-perl 4.59-4
> retitle 593043 Recent change broke publican
> affects 593043 publican
> thanks
>
> On Sun, 15 Aug 2010, Lucas Nussbaum wrote:
>> During a rebuild of all packages in sid, your package failed to build on
>> amd64.
>>
>> Relevant part:
>>> make[1]: Entering directory
>>> `/build/user-publican_2.1-2-amd64-k4WX4i/publican-2.1'
>>> dh_auto_build
>>> Copying lib/Publican/TreeView.pm -> blib/lib/Publican/TreeView.pm
>>> Copying lib/Publican/CreateBook.pm -> blib/lib/Publican/CreateBook.pm
>>> Copying lib/Publican.pm -> blib/lib/Publican.pm
>>> Copying lib/Publican/XmlClean.pm -> blib/lib/Publican/XmlClean.pm
>>> Copying lib/Publican/Translate.pm -> blib/lib/Publican/Translate.pm
>>> Copying lib/Publican/CreateBrand.pm -> blib/lib/Publican/CreateBrand.pm
>>> Copying lib/Publican/WebSite.pm -> blib/lib/Publican/WebSite.pm
>>> Copying lib/Publican/Localise.pm -> blib/lib/Publican/Localise.pm
>>> Copying lib/Publican/Builder.pm -> blib/lib/Publican/Builder.pm
>>> Copying bin/publican -> blib/script/publican
>>> [31m
>>> DEBUG: [0m[31mPublican: config loaded
>>> [0mSetting up ar-SA
>>> mkdir /ar-SA: Permission denied at lib/Publican/Builder.pm line 251
>>> dh_auto_build: perl Build returned exit code 13
>>> make[1]: *** [override_dh_auto_build] Error 2
>
> This used to work. After some investigation, I discovered that the version
> 4.59-4 of libconfig-simple-perl broke it, 4.59-3 worked fine.
>
> It looks like the last upload also activated the patch "empty_value.patch"
> which was not applied in the former upload (even if present in the
> tarball). That patch changes the behaviour in a way that is not really
> desirable apparently. The module does no longer return "undef" for
> non-existent entries but an empty value which leads publican to believe
> that the value is set by the user and to use "" instead of its default
> value ("tmp").
>
> We should not diverge on this point without upstream backing. I suggest
> to drop that patch... it's clearly a behavioural change that has
> unexpected impact.
>
> Disabling that patch make publican work again.Hi Raphael, The upstream appears to be dead, and there are other (Debian) users of libconfig-simple-perl who requested this change. I agree that either behavior is arbitrary, but since it can't behave both ways, would you be receptive to a patch for publican that treats the empty value as the default? Thank you, Tony
signature.asc
Description: OpenPGP digital signature
