[pacman-dev] Pacman Hooks
dpmcgee at gmail.com
Sat Jan 29 15:30:10 EST 2011
On Sat, Jan 29, 2011 at 2:20 PM, Sascha Kruse <knopwob at googlemail.com> wrote:
> 2011/1/29 Dan McGee <dpmcgee at gmail.com>:
>> On Sat, Jan 29, 2011 at 8:36 AM, Sascha Kruse <knopwob at googlemail.com> wrote:
>>> - The "File" argument can't begin with a / because it is matched
>>> against alpm_pkg_get_data and that
>>> file-strings don't have the leading /
>> This is a must-have, your logic is backwards. We never assume the
>> install root is /; file paths in all install scripts should be written
>> this way too, not to mention everywhere in the library. So nothing
>> wrong here, it would be wrong if you did any prefix whatsoever.
> I get your point, but i think it would be confusing for the users.
> I would intuitively write File = /usr/share/fonts/*
> instead of File = usr/share/fonts/*.
> So maybe we should accept both and in the latter case ignore
> the leading / ?
Maybe I wasn't clear.
We will *not* take a patch that prefixes paths with anything,
anywhere, anytime. pactest is a perfect example of relative dbpaths
and dbroots that this needs to work for.
More information about the pacman-dev