search

andrewgregory / pacutils

Helper library for libalpm based programs.
MIT License
107 stars 17 forks source link

Support pacman 7.0 #76

Open Foxboron opened 2 weeks ago

Foxboron commented 2 weeks ago

pacconf returns an error, but I suspect there are other places as well. Needs to support DisableSandbox as well I assume?

warning: config /etc/pacman.conf line 39: unknown option 'DownloadUser'
andrewgregory commented 2 weeks ago

Please check with the pacman-7.0.0 branch.

christian-heusel commented 2 weeks ago

While the pacman-7.0.0 branch silences the warning about the unknown config option, it also does not correctly parse/display their value:

$ make clean
$ make SYSCONFDIR=/etc LOCALSTATEDIR=/var -j$(nproc)
$ LD_LIBRARY_PATH=lib/ ./src/pacconf --config=pacman.conf | grep -E "DisableSandbox|DownloadUser"
<no result>
$ LD_LIBRARY_PATH=lib/ ./src/pacconf --config=pacman.conf DisableSandbox DownloadUser
warning: unknown directive 'DisableSandbox'
warning: unknown directive 'DownloadUser'
$ grep -E "DisableSandbox|DownloadUser" pacman.conf
DownloadUser = alpm
DisableSandbox

I have sent a PR to fix it 😅

solsticedhiver commented 1 week ago

well, we need a new release with the fixes. pacman 7 is out.