gnome wayland下,通过terminal启动可以正常截屏,但是通过启动器打开无法获取到截屏,是什么原因造成的呢
Search Criteria
Package Details: flameshot-git r1938.fa29bcb4-1
Package Actions
Git Clone URL: | https://aur.archlinux.org/flameshot-git.git (read-only, click to copy) |
---|---|
Package Base: | flameshot-git |
Description: | Powerful yet simple to use screenshot software |
Upstream URL: | https://github.com/flameshot-org/flameshot |
Keywords: | screenshot |
Licenses: | GPL |
Conflicts: | flameshot |
Provides: | flameshot |
Submitter: | j1simon |
Maintainer: | mehrad (borgman_jeremy) |
Last Packager: | mehrad |
Votes: | 41 |
Popularity: | 0.186911 |
First Submitted: | 2017-07-08 16:07 (UTC) |
Last Updated: | 2023-11-10 14:16 (UTC) |
Dependencies (11)
- hicolor-icon-theme (hicolor-icon-theme-gitAUR)
- kguiaddons5
- qt5-base (qt5-base-gitAUR, qt5-base-headlessAUR)
- qt5-svg (qt5-svg-gitAUR)
- cmake (cmake-gitAUR) (make)
- git (git-gitAUR, git-glAUR) (make)
- qt5-tools (make)
- grim (grim-gitAUR) (optional) – To make Sway, Hyprland, and wlroots work
- xdg-desktop-portal (xdg-desktop-portal-gitAUR) (optional) – To make Sway, Hyprland, and wlroots work
- xdg-desktop-portal-kde (xdg-desktop-portal-kde-gitAUR) (optional) – For KDE Plasma
- xdg-desktop-portal-wlr (xdg-desktop-portal-wlr-gitAUR) (optional) – To make Sway, Hyprland, and wlroots work
Required by (4)
- cutetranslation-flameshot (requires flameshot)
- dmenu_shot-git (requires flameshot)
- rabbithole (requires flameshot)
- screenshot-bash (requires flameshot) (optional)
Sources (1)
mnsd commented on 2024-05-14 02:23 (UTC)
earlybird commented on 2023-11-12 05:20 (UTC)
add kguiaddons5 to the depends list please
xiota commented on 2023-11-10 16:32 (UTC) (edited on 2023-11-10 16:36 (UTC) by xiota)
Package should be built in clean chroot before committing updates.
Would you please elaborate more. I'm interested to know if this can also be automated.
Elaboration is on Arch Wiki: Building in a clean chroot. The command you need for your Makefile is described there.
pkgver()
needs to be updated to#.r#.g#
format, withoutv
prefix. See VCS package guidelinesIn ... you can find exactly the line we have used in flameshot-git AUR. Therefore,not only we are on par with the guideline, but the code we have is verbatim copy of the sample the AUR guideline provided.
VCS package guidelines, 2.6.1 Git lists options in order of preference. The format #.r#.g#
is clearly preferred. The format you have chosen has condition for use: "If there are no tags..." Since the project does have tagged released, the package version should be in the preferred format #.r#.g#
.
Versions in this format #.r#.g#
are more useful than r#.#
because it immediately communicates where a particular build is tracking with respect to the releases.
Tagging on your project frequently occurs outside the master branch. So it does take more effort than git describe
to obtain a useful version string.
_tag=$(git tag | grep -Ev '[a-z]{2}' | sort -V | tail -1)
_revision=$(git rev-list --count $_tag..HEAD)
_commit=$(git rev-parse --short=7 HEAD)
printf '%s.r%s.g%s' "${_tag#v}" "$_revision" "$_commit"
Remove irrelevant comments: first-line "mode", "Contribution Instructions", "version numbers".
...
Those comments appear to have been copied/pasted from some old tutorial. A portion does not even appear relevent to this PKGBUILD. They are better moved to a different document or replaced with links to relevant ArchWiki pages.
Do not use
-j$(nproc...)
. Users set this themselves in makepkg.conf.There is no mention in the guideline that indicates this is wrong.
Regardless of whether there is a specific guideline, overriding user preferences is disrespectful. Rather than force all users to use a specific setting, users could be instructed on how to configure makepkg.
Instead of teaching less technical users... since it is AUR, technical users can modify it to their liking before installing.
If I took the "instead of teaching" approach, your PKGBUILD would still be unbuildable.
Sure, experienced "technical users" can fix packages before installing, but packages that are well maintained don't have to be fixed.
What did you expect "less technical users" to do with a broken PKGBUILD with numerous defects that would not build for over 1.5 years?
Rolv commented on 2023-11-10 14:51 (UTC)
Awesome, installation from the AUR now working flawlessly (at least for me), thanks @mehrad
mehrad commented on 2023-11-10 14:42 (UTC) (edited on 2023-11-10 14:53 (UTC) by mehrad)
Thanks to @Vedun, @Rolv, @Kage-Yami, @xiota, and @etrigan63 for the info, suggestions and elaborations. I updated the PKGBUILD and SRCINFO accordingly. Also I added a part in Makefile to handle SRCINFO generation to avoid some of the raised issues.
Regarding @xiota comment:
Thanks for the detailed constructive comment and suggestuions.
Provides should be for flameshot, not
flameshot-git
.
Thanks, fixed.
kguiaddons5
should be depends, not makedepends. This was reported back in Feb 2022.
Fixed.
Package should be built in clean chroot before committing updates.
Would you please elaborate more. I'm interested to know if this can also be automated.
.SRCINFO
should be generated before committing updates. See AUR submission guidelines
Thanks, noted, and Makefile thanks to your comment now handles this too.
pkgver()
needs to be updated to#.r#.g#
format, withoutv
prefix. See VCS package guidelines
In the VCS package guidelines they literally state "See PKGBUILD-vcs.proto for generic examples showing the intended output." and when you click on the link and scroll down to line 49 you can find exactly the line we have used in flameshot-git AUR. Therefore,not only we are on par with the guideline, but the code we have is verbatim copy of the sample the AUR guideline provided.
Remove irrelevant comments: first-line "mode", "Contribution Instructions", "version numbers".
The first line is not a comment although it is in form of a comment, and the contribution instructions are there to help new and/or less techie users make sense of what is going on, and to help them know how to contribute. If you don't find those comments relevant, feel free to treat them like the interpreter: as comments :)
Do not use
-j$(nproc...)
. Users set this themselves in makepkg.conf.
There is no mention in the guideline that indicates this is wrong. I agree that people can define MAKEFLAGS
variable in their makepkg.conf
, but the reality is that they don't, which means we get complaints all the time that the build time is long. Instead of teaching less technical users, we went the other rout of setting a relatively high core count, but since it is AUR, technical users can modify it to their liking before installing. We already spare one core for the rest of the system to function without issues during the build process.
Recommend to remove @borgman_jeremy from comaintainers list. User has no other packages and has no AUR activity since Sep 2020.
Thanks for your concern. Jeremy stepped down as the head dev of Flameshot some months back, but he has not stepped down/away from maintaining this repo, therefore, he is and will be listed as long as he is interested in co-maintaining this repo.
etrigan63 commented on 2023-11-09 17:49 (UTC)
It appears you are missing a dependency. You need to add extra/kguiaddons5
explicitly as many window managers do not install it by default. hyprland
being one of them.
xiota commented on 2023-11-09 11:53 (UTC) (edited on 2023-11-09 19:22 (UTC) by xiota)
- Provides should be for
flameshot
, notflameshot-git
. kguiaddons5
should be depends, not optdepends. This was reported back in Feb 2022.- Package should be built in clean chroot before committing updates.
.SRCINFO
should be generated before committing updates. See AUR submission guidelinespkgver()
needs to be updated to#.r#.g#
format, withoutv
prefix. See VCS package guidelines- Remove irrelevant comments: first-line "mode", "Contribution Instructions", "version numbers".
- Do not use
-j$(nproc...)
. Users set this themselves inmakepkg.conf
. - Recommend to remove @borgman_jeremy from comaintainers list. User has no other packages and has no AUR activity since Sep 2020.
Kage-Yami commented on 2023-10-30 10:20 (UTC)
It appears that kguiaddons5
is a mandatory make
dependency - attempting to build in a clean chroot with aurto
fails. However, building with yay
(when I've got kguiaddons5
already installed on my system) does succeed.
Full error from chroot build
CMake Warning at src/CMakeLists.txt:14 (find_package):
By not providing "FindKF5GuiAddons.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"KF5GuiAddons", but CMake did not find one.
Could not find a package configuration file provided by "KF5GuiAddons" with
any of the following names:
KF5GuiAddonsConfig.cmake
kf5guiaddons-config.cmake
Add the installation prefix of "KF5GuiAddons" to CMAKE_PREFIX_PATH or set
"KF5GuiAddons_DIR" to a directory containing one of the above files. If
"KF5GuiAddons" provides a separate development package or SDK, be sure it
has been installed.
Flameshot predefined color palette large: false
-- Found Git: /usr/bin/git (found version "2.42.0")
git found: /usr/bin/git in version 2.42.0
FLAMESHOT_GIT_HASH: fa29bcb4
-- Configuring done (0.6s)
CMake Error at src/CMakeLists.txt:217 (target_link_libraries):
Target "flameshot" links to:
KF5::GuiAddons
but the target was not found. Possible reasons include:
* There is a typo in the target name.
* A find_package call is missing for an IMPORTED target.
* An ALIAS target is missing.
-- Generating done (0.0s)
CMake Generate step failed. Build files cannot be regenerated correctly.
Rolv commented on 2023-10-26 13:01 (UTC) (edited on 2023-10-27 12:15 (UTC) by Rolv)
I'm also unable to get this package to build. Says I am missing kguiaddons
but I checked and shouldn't the package be kguiaddons5
?
EDIT: can confirm, after editing the PKGBUILD
and changing kguiaddons
to kguiaddons5
, package builds and installs correctly.
For anyone else who wants to install, this worked for me: grab the PKGBUILD
from the top right of this page under Package Actions
and put it in a random, empty directory. Edit it and just add a 5
after kguiaddons
. Then, in that directory run makepkg
, then sudo pacman -U flameshot-git-r1938.fa29bcb4-1-x86_64.pkg.tar.zst
Vedun commented on 2023-10-05 05:49 (UTC)
The package is not being assembled.
Output in the terminal:
...
CMake Deprecation Warning at external/Qt-Color-Widgets/color_widgets_designer_plugin/CMakeLists.txt:17 (cmake_minimum_required):
Compatibility with CMake < 3.5 will be removed from a future version of
CMake.
Update the VERSION argument <min> value or use a ...<max> suffix to tell
CMake that the project does not need compatibility with older versions.
CMake Warning at src/CMakeLists.txt:14 (find_package):
By not providing "FindKF5GuiAddons.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"KF5GuiAddons", but CMake did not find one.
Could not find a package configuration file provided by "KF5GuiAddons" with
any of the following names:
KF5GuiAddonsConfig.cmake
kf5guiaddons-config.cmake
Add the installation prefix of "KF5GuiAddons" to CMAKE_PREFIX_PATH or set
"KF5GuiAddons_DIR" to a directory containing one of the above files. If
"KF5GuiAddons" provides a separate development package or SDK, be sure it
has been installed.
Flameshot predefined color palette large: false
-- Found Git: /usr/bin/git (found version "2.42.0")
git found: /usr/bin/git in version 2.42.0
FLAMESHOT_GIT_HASH: ad390603
-- Configuring done (4.5s)
CMake Error at src/CMakeLists.txt:217 (target_link_libraries):
Target "flameshot" links to:
KF5::GuiAddons
but the target was not found. Possible reasons include:
* There is a typo in the target name.
* A find_package call is missing for an IMPORTED target.
* An ALIAS target is missing.
...
Pinned Comments
GC268DM commented on 2022-05-29 10:40 (UTC)
What worked for me was
Where -a stands for aur and --devel for development packages.
Since the --devel flag checks if behind the development packages (like "*-git" packages) there are any newer versions, it will update to the latest version of the package in git and not require and update on the aur listing
mehrad commented on 2022-01-19 10:26 (UTC) (edited on 2022-01-19 10:28 (UTC) by mehrad)
Please stop flagging this AUR as out-of-date unless you have tried these:
If you use a proper AUR helper (e.g
paru
,yay
) this AUR will be use the latest commit of the Github repo. If you usepamac
, it will not update based on the latest commit automatically and you should dopamac install --upgrade flameshot-git
to force it to check for upgrades. Therefore, this AUR cannot become out-of-date unless the Github URL or dependencies get out of date.