On Sat, Jan 29, 2011 at 2:20 PM, Sascha Kruse <knopwob@googlemail.com> wrote:
2011/1/29 Dan McGee <dpmcgee@gmail.com>:
- 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
On Sat, Jan 29, 2011 at 8:36 AM, Sascha Kruse <knopwob@googlemail.com> wrote: 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