On 18/10/15 01:06, Andrew Gregory wrote:
On 10/17/15 at 07:50pm, Allan McRae wrote:
Signed-off-by: Allan McRae <allan@archlinux.org> --- src/pacman/files.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Surely, if anything, we should be adding handle->root, not just a raw slash. I'm not sure it makes sense to do that for sync packages though. All of our path handling for local packages uses absolute paths because we're referring to actual objects on the filesystem. `pacman -Qo` doesn't even allow paths that don't actually exist. Sync packages, on the other hand, are just a list of strings in a database with no correlation to the filesystem. I'm not sure something like `pacman -Fo /foo/bar` even makes sense, and `pacman --root=/baz -Fo /foo/bar` and `pacman --root=/baz -Fo /baz/foo/bar` make even less.
I would say to just treat sync file lists as raw strings and forget that they represent paths at all.
Agreed. Printing without any root prefix is the most consistent. I'll submit a new patch that consistently does not print and root prefix. Allan