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

azote: update to 1.8.1 #25903

Closed
wants to merge 1 commit into from
Closed

azote: update to 1.8.1 #25903

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 26, 2020

version=1.7.10
revision=2
version=1.8.1
revision=1
build_style=python3-module
hostmakedepends="python3-setuptools"
depends="python3>=3.4 python3-setuptools python3-gobject python3-Pillow gtk+3 feh python3-send2trash xrandr grim slurp ImageMagick python3-yaml"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please drop the following packages from the dependencies:

  • xrandr
  • feh
  • python3-send2trash
  • grim
  • slurp
  • ImageMagick
  • python3-yaml
  • python3-setuptools

All but the last enable optional functionality, and the program itself provides a warning about missing packages and disabled functionality when users run without these packages installed. There is no need to pull in everything by force, especially the xrandr dependency for Wayland users, or the grim and slurp dependencies for Xorg users.

The last dependency is strictly wrong, as setuptools is not needed anywhere for runtime operation of azote.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated, thanks!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, azote uses pkg_resources module from setuptools.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My mistake, python3-setuptools definitely belongs in depends because of pkg_resources. Thanks @Chocimier.

This means python3 can just be dropped, because python3-setuptools already pulls that in.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated, thanks @Chocimier and @ahesford!

@ghost ghost requested a review from ahesford October 26, 2020 19:26
srcpkgs/azote/template Show resolved Hide resolved
build_style=python3-module
hostmakedepends="python3-setuptools"
depends="python3>=3.4 python3-setuptools python3-gobject python3-Pillow gtk+3 feh python3-send2trash xrandr grim slurp ImageMagick python3-yaml"
depends="python3>=3.4 python3-gobject python3-Pillow gtk+3"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please drop >=3.4, it's too old to work with like anything.

Copy link
Author

@ghost ghost Oct 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced python3 with python3-setuptools


post_install() {
vmkdir usr/bin
vmkdir usr/share/${pkgname}
vmkdir usr/share/applications

vcopy "dist/${pkgname}.svg" usr/share/${pkgname}
vcopy "dist/${pkgname}-arch" usr/share/${pkgname}
vcopy "dist/${pkgname}" usr/share/${pkgname}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's no value in copying this script to usr/share and then symlinking to it from usr/bin, just vcopy right into usr/bin and drop the symlink in line 26.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@ghost ghost requested a review from ahesford October 27, 2020 15:41
@ahesford ahesford closed this in 856d631 Oct 29, 2020
Logarithmus pushed a commit to Logarithmus/void-packages that referenced this pull request Nov 15, 2020
Closes: void-linux#25903 [via git-merge-pr]
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants