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

/usr/pixmaps polluting #283

Open
kuraga opened this issue Oct 1, 2024 · 4 comments
Open

/usr/pixmaps polluting #283

kuraga opened this issue Oct 1, 2024 · 4 comments

Comments

@kuraga
Copy link
Contributor

kuraga commented Oct 1, 2024

$ make
<...>
$ make DESTDIR="image" install-system
<...>
$ find image/usr/pixmaps/
image/usr/pixmaps/
image/usr/pixmaps/tags.png
image/usr/pixmaps/edit_add.png
image/usr/pixmaps/edit.png
image/usr/pixmaps/add.xcf
image/usr/pixmaps/mnemosyne_small.png
image/usr/pixmaps/mnemosyne.ico
image/usr/pixmaps/mnemosyne.icns
image/usr/pixmaps/exit.png
image/usr/pixmaps/image.png
image/usr/pixmaps/statistics.png
image/usr/pixmaps/bold.png
image/usr/pixmaps/highlight.png
image/usr/pixmaps/deck.xcf
image/usr/pixmaps/filesave.png
image/usr/pixmaps/filenew.png
image/usr/pixmaps/edit.xcf
image/usr/pixmaps/filesaveas.png
image/usr/pixmaps/sound.png
image/usr/pixmaps/delete.png
image/usr/pixmaps/configure.png
image/usr/pixmaps/fileopen.png
image/usr/pixmaps/cards.png
image/usr/pixmaps/cancel.png
image/usr/pixmaps/mnemosyne.png
image/usr/pixmaps/italic.png
image/usr/pixmaps/latex.png
image/usr/pixmaps/deck.png
image/usr/pixmaps/underline.png
image/usr/pixmaps/delete.xcf
image/usr/pixmaps/mnemosyne_large.png
image/usr/pixmaps/mnemosyne.svg
image/usr/pixmaps/cards.xcf
image/usr/pixmaps/add.png

I don't know the rules of /usr/pixmaps polluting but writing some "global" files such as /usr/pixmaps/cancel.png is odd...

@kuraga
Copy link
Contributor Author

kuraga commented Oct 1, 2024

I guess they should be at /usr/share/pixmaps/mnemosyne.

@kuraga
Copy link
Contributor Author

kuraga commented Oct 3, 2024

?

mnemosyne/setup.py

Lines 109 to 112 in cf6e8b0

# Pixmaps.
for pixmap in os.listdir("pixmaps"):
data_files.append(("pixmaps",
[os.path.join("pixmaps", pixmap)]))

@kuraga
Copy link
Contributor Author

kuraga commented Oct 3, 2024

@pbienst
Copy link
Contributor

pbienst commented Oct 3, 2024

I not able to devote time to Mnemosyne these days, but if you care enough about this, I'm happy to accept patches.

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

2 participants