Skip to content

Handling DistributionNotFound exceptions#60

Draft
sezanzeb wants to merge 285 commits intokassoulet:mainfrom
sezanzeb:flatpak-fix-pkg_resources
Draft

Handling DistributionNotFound exceptions#60
sezanzeb wants to merge 285 commits intokassoulet:mainfrom
sezanzeb:flatpak-fix-pkg_resources

Conversation

@sezanzeb
Copy link
Copy Markdown
Contributor

@sezanzeb sezanzeb commented Jan 6, 2022

@jmaibaum does this work for you? Is the os.environ.get('container') == 'flatpak' test reliable?

sezanzeb and others added 30 commits February 12, 2019 23:30
Code Cleaning and Invalid Audiofiles Handling
Fixed disc number interpretation
Closes lp:1819223
Closes lp:1851147, thanks phil995511
The basefolder is not quoted as it should be.
If the converted file is within a subdirectory containing for example a # sign,
soundconverter would hang.
Externalize TargetNameGenerator.safe_name()
@sezanzeb sezanzeb changed the title Can handle DistributionNotFound exceptions Handling DistributionNotFound exceptions Jan 6, 2022
@jmaibaum
Copy link
Copy Markdown
Contributor

jmaibaum commented Jan 7, 2022

@sezanzeb Thanks for looking into this! A flatpak built from this PR's ref launches normally again, but it shows unknown-version instead of a proper version number. Both on the command line:

$ flatpak run --user org.soundconverter.SoundConverter//master
ERROR: Disabling mp3-id-tags output. Do you have "gst-plugins-good" installed?
ERROR: Disabling mp3-vbr-tags output. Do you have "gst-plugins-ugly" installed?
soundconverter unknown-version

and in the GUI:

Bildschirmfoto von 2022-01-07 21-37-57

Is it possible to specify the version at build time?

@sezanzeb
Copy link
Copy Markdown
Contributor Author

sezanzeb commented Jan 7, 2022

"unknown-version" is the fallback for this case

Is it possible to specify the version at build time?

Yes, input-remapper does something similar with the commit hash: https://github.com/sezanzeb/input-remapper/blob/main/setup.py#L34, it writes it into a python file which is then imported during runtime. Instead of asking git tag or something we could also use a constant in setup.py with the version, use it in the DistUtilsExtra.auto.setup configuration and use it to generate a file that can be imported during runtime.

Or we hardcode the version number. It is hardcoded in setup.py already anyway. We just shouldn't forget to change it everywhere.

@jmaibaum
Copy link
Copy Markdown
Contributor

jmaibaum commented Jan 9, 2022

I have no preference if you hardcode the version number or if I have to set it during packaging, but I'd prefer to have something else than "unknown version" showing up.

jmaibaum added a commit to jmaibaum/org.soundconverter.SoundConverter that referenced this pull request Jan 9, 2022
This should not be necessary, but until someone can tell me what is
going on here, at least this starts soundconverter again.

Revisit after kassoulet/soundconverter#60 has
been merged with a way to set a proper version number.

Fixes flathub#6
@sezanzeb sezanzeb marked this pull request as draft January 17, 2022 11:39
@sezanzeb
Copy link
Copy Markdown
Contributor Author

sezanzeb commented Jan 17, 2022

I hope I'll find time to do this at some point, there are a lot of other things open in other projects right now

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

Successfully merging this pull request may close these issues.

10 participants