[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#528021: maintainer scripts put files in /usr/local/ (policy 9.1.2)



Norbert Preining <preining@logic.at> wrote:

> I guess what we have here are *two* bugs so we should duplicate them:
> 1) dh_installtex generated code
> 2) missing try of removing /u/l/s/texmf
> 3) trigger action
> (that are three, I know, but it is late and I had loads of good wine!)
>
> 1) fixed since ages

ACK

> 2) fixed in tex-common trunk, but nobody commented on my patch for now

Disregarding comment's, that's just 

-rm -f /usr/local/share/texmf/ls-R 2>/dev/null || true
+rmdir --ignore-fail-on-non-empty /usr/local/share/texmf/ 2>/dev/null || true

also reading the bug that made us add the original line, #333308,
doesn't give any reason why this should have been right.

By the way, in this bug log I read:

,----
| From: Frank Küster <frank@kuesterei.ch>
| To: Lars Wirzenius <liw@iki.fi>
| 
| > postrm can't rely on non-essential packages when being called during
| > purge (policy 7.2). Specifically, dpkg gives no guarantee that any
| > packages being depended on exist anymore on the system, meaning that ucf
| > might not be there. 
| 
| This we know, and therefore we do
| 
| purge_ucf_file () {
|   local conffile="$1"
|   purge_file $conffile
|   if [ -x /usr/bin/ucf ]; then
|     ucf --purge "${conffile}"
|   fi
| }
`----

Hm, remembering this could have saved us quite a lot. I'm also surprised
that it didn't show up earlier, since Lars also writes:

> This is what happens to happen with piuparts (well,
> I tried to make piuparts cause it to happen). 

> 3) needs fixing, I have to find a good way to do that ... 

I leave trigger stuff for stable to you until I feel comfortable with
the code.

Regards, Frank

-- 
Dr. Frank Küster
Debian Developer (TeXLive)
VCD Aschaffenburg-Miltenberg, ADFC Miltenberg
B90/Grüne KV Miltenberg


Reply to: