Package: fvwm
Version: 2.5.12-4
Severity: important

Hello Manoj,

I think there are 2 oversight in fvwm postinst: first

if [ ! -x /etc/menu-methods/fvwm ]; then
    chmod a+x /etc/menu-methods/fvwm
fi

According to Debian policy, users are well within theirs right to remove the
file and this will be preserved on upgrade by dpkg. In that case the
code above will break. You should rather do
if [ -e /etc/menu-methods/fvwm ]

Also later on ,
    # Activate menu-methods script
        chmod a+x /etc/menu-methods/${package_name}

This is redundant and has the same problem.

I think the postrm is fine.

Cheers,
-- 
Bill. <[EMAIL PROTECTED]>

Imagine a large red swirl here. 


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to