Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nouveau.yaml is overwriten #177

Closed
jonasplaum opened this issue Mar 21, 2025 · 5 comments
Closed

Nouveau.yaml is overwriten #177

jonasplaum opened this issue Mar 21, 2025 · 5 comments

Comments

@jonasplaum
Copy link
Contributor

jonasplaum commented Mar 21, 2025

While updating from nouveau 3.4.2 our config in /opt/nouveau/etc/nouveau.yaml gets overwritten on debian12.

I tested this on a clean debian12 system with the following commands (The jfrog repo is already added):

apt install couchdb=3.4.2~bookworm
apt install couchdb-nouveau=3.4.2~bookworm

change something like the index path in /opt/nouveau/etc/nouveau.yaml

apt install couchdb=3.4.3~bookworm
apt install couchdb-nouveau=3.4.3~bookworm

The changes to the config file are lost.

@nickva
Copy link
Contributor

nickva commented Mar 21, 2025

Thank you @jonasplaum. That certainly looks like an issue to fix.

@jonasplaum
Copy link
Contributor Author

jonasplaum commented Mar 24, 2025

Thanks for the fix, but make build-couch $(lsb_release -cs) PLATFORM=$(lsb_release -cs) fails for me with the following message after this patch. Probably because this file is used in the couchdb package (but the config file is not in this package) and not in couchdb-nouveau. Maybe there is one per package needed.

dpkg-deb: building package 'couchdb-dbgsym' in '../couchdb-dbgsym_3.4.3-3664cb0~bookworm_amd64.deb'.
dpkg-deb: building package 'couchdb-nouveau' in '../couchdb-nouveau_3.4.3-3664cb0~bookworm_amd64.deb'.
dpkg-deb: error: conffile '/opt/nouveau/etc/nouveau.yaml' does not appear in package
dh_builddeb: error: dpkg-deb -Zgzip --build debian/couchdb .. returned exit code 2
dh_builddeb: error: Aborting due to earlier error

@rnewson rnewson reopened this Mar 24, 2025
@rnewson
Copy link
Member

rnewson commented Mar 26, 2025

can you test again please (#181)

@jonasplaum
Copy link
Contributor Author

Tested again after the merge. It works as expected.

@nickva
Copy link
Contributor

nickva commented Mar 27, 2025

Thank you!

I think we can close the issue

@nickva nickva closed this as completed Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants