Welcome to the Pacman bug collection. Please search the current bugs and feature requests before filing a new one! Use advanced search and select "Search in Comments".

* Please select the correct category and version.
* Write a descriptive summary, background info, and provide a reproduceable test case whenever possible.
Tasklist

FS#7182 - pacman -Q --changelog works only as root

Attached to Project: Pacman
Opened by Attila (attila) - Thursday, 17 May 2007, 02:18 GMT-5
Last edited by Xavier (shining) - Monday, 03 December 2007, 14:52 GMT-5
Task Type Bug Report
Category Backend/Core
Status Closed
Assigned To Aaron Griffin (phrakture)
Dan McGee (toofishes)
Architecture All
Severity Low
Priority Normal
Reported Version 3.0.4
Due in Version 3.1.0
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

I created an own package using fakeroot (umask user: 007) and includes a ChangeLog. As normal user i can't see it with pacman (Fehler: Kein Änderungsprotokoll für 'PACKAGE' verfügbar) and as root it works. The reason be the file permissions of this Changelog:
-rw-r----- 1 root root

An "find /var/lib/pacman/local -type f -perm 640" shows that the *.install files from my own packages has the same file permissions but this has for me more cosmetically character.
This task depends upon

View Dependency Graph

This task blocks these from closing
 FS#8109 - Pacman 3.1 Release Roadmap 
Closed by  Xavier (shining)
Monday, 03 December 2007, 14:52 GMT-5
Reason for closing:  Fixed
Additional comments about closing:  fixed by Aaron in git, commit 5f811dc020.
Comment by Aaron Griffin (phrakture) - Thursday, 17 May 2007, 11:31 GMT-5
This is technically a packaging problem, but we can fix it (and probably should) in libalpm.

Note to self: on DB extraction, use archive_entry_set_mode() and force DB to 644 across the board.
Comment by Aaron Griffin (phrakture) - Wednesday, 14 November 2007, 20:12 GMT-5
This is done locally, need to test it though
Comment by Aaron Griffin (phrakture) - Wednesday, 14 November 2007, 20:22 GMT-5
http://code.phraktured.net/?p=pacman.git;a=commitdiff;h=2d690c6633cb590f590410c70efc21ae937ce572
Comment by Dan McGee (toofishes) - Thursday, 29 November 2007, 14:04 GMT-5
I believe this is working fine now, but can anyone confirm? Perhaps make a package with a changelog with permissions of 400, then install and see if you can read the changelog as root.
Comment by Aaron Griffin (phrakture) - Thursday, 29 November 2007, 14:19 GMT-5
Should be covered by: http://projects.archlinux.org/git/?p=pacman.git;a=commit;h=6e8daa553bbd50ec541add9549bfd95030c335f5
Comment by Xavier (shining) - Thursday, 29 November 2007, 14:32 GMT-5
Looks like the last pacman package in Dan's repo was built after this change.
I can't confirm it was fixed though, the changelog kept the 400 permission when installing the package.
Sorry, I don't have time to investigate this issue now, but tomorrow or this week end, I can have a better look.

To Dan : I think you meant "as user", not "as root" :)
Comment by Xavier (shining) - Thursday, 29 November 2007, 18:45 GMT-5
Well, I gave a quick look, and it's actually pretty simple.
_alpm_unpack in util.c is only used for extracting the scriptlets and the databases.
The ChangeLog extraction is done in add.c , with every other files, and doesn't use the _alpm_unpack function.
Comment by Aaron Griffin (phrakture) - Thursday, 29 November 2007, 18:55 GMT-5
Already done:
http://code.phraktured.net/?p=pacman.git;a=commit;h=531772693e0aade8a577f33af3e3c91aac616d84

See also the commit above it for why I used 0644 for both. I need to test THAT commit much much more.
Comment by Xavier (shining) - Thursday, 29 November 2007, 19:39 GMT-5
Ok, just confirming that commit does fix the changelog permission problem.

Loading...