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

Snap packaging improvements #207

Merged
merged 4 commits into from
Mar 29, 2019
Merged

Conversation

brlin-tw
Copy link
Contributor

No description provided.

…terfaces (mikf#197)

* FIXME no longer applied
* Obsoleted HOME environment variable assignment

Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
The selective-pull stage snap is superseded by selective-checkout, prefer the new one.

Refer-to: Selective-checkout: Check out the tagged release revision if it isn't promoted to the stable channel <https://forum.snapcraft.io/t/the-selective-pull-scriptlet-stage-snap-workaround/10389>
Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
Refer-to: Scriptlets <https://docs.snapcraft.io/scriptlets/4892>
Refer-to: Tab completion for snaps <https://docs.snapcraft.io/tab-completion-for-snaps/2261>
Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
This patch ensures that the user will be acknowledge the missing
connection to the `removable-media` interface.

Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
@mikf mikf merged commit 81d4d49 into mikf:master Mar 29, 2019
@brlin-tw brlin-tw deleted the snap-packaging-improvements branch March 29, 2019 09:38
@brlin-tw
Copy link
Contributor Author

brlin-tw commented Mar 29, 2019

I'm aware of the breakage of the snap building due to commit 6612081 and is working on fixing it.

@mikf
Copy link
Owner

mikf commented Mar 29, 2019

...
  File "/build/gallery-dl/parts/gallery-dl/build/gallery_dl/extractor/common.py", line 17, in <module>
    import requests
ModuleNotFoundError: No module named 'requests'
make: *** [docs/supportedsites.rst] Error 1
Makefile:29: recipe for target 'docs/supportedsites.rst' failed
Failed to run 'override-build': Exit code was 2.

Building the unnecessary list of supported sites failes because requests isn't installed at this point in time. Changing make to make gallery-dl.bash_completion should work here, since building the bash completion script doesn't import any external Python packages.

brlin-tw added a commit to brlin-tw/gallery-dl that referenced this pull request Mar 29, 2019
The build failed due to missing `requests` build dependency, this patch
drops the unused component to build to avoid the problem.

The manpages are still built for the upcoming read-manual workaround.

Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
mikf pushed a commit that referenced this pull request Mar 29, 2019
The build failed due to missing `requests` build dependency, this patch
drops the unused component to build to avoid the problem.

The manpages are still built for the upcoming read-manual workaround.

Signed-off-by: 林博仁(Buo-ren Lin) <Buo.Ren.Lin@gmail.com>
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.

None yet

2 participants