On Sat, Jan 29, 2011 at 2:20 PM, Sascha Kruse <[email protected]> wrote: > 2011/1/29 Dan McGee <[email protected]>: >> On Sat, Jan 29, 2011 at 8:36 AM, Sascha Kruse <[email protected]> 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. -Dan
