Package Details: cropgui 0.7-1

Git Clone URL: https://aur.archlinux.org/cropgui.git (read-only, click to copy)
Package Base: cropgui
Description: a simple tool for lossless jpeg cropping
Upstream URL: https://github.com/jepler/cropgui
Licenses: GPL
Submitter: jose1711
Maintainer: jose1711
Last Packager: jose1711
Votes: 14
Popularity: 0.000000
First Submitted: 2010-05-15 21:14 (UTC)
Last Updated: 2023-11-30 08:19 (UTC)

Latest Comments

1 2 3 4 5 6 Next › Last »

kovdan01 commented on 2023-11-30 16:16 (UTC) (edited on 2023-11-30 16:18 (UTC) by kovdan01)

Hello @jose1711 and thanks for maintaining the package! Currently, there is an issue described in the commit message below. The patch below resolves that. Could you please apply the commit? Thanks!

From 977bd569accc3edff6bb9acc08a4483336a3b5a6 Mon Sep 17 00:00:00 2001
From: Daniil Kovalev <daniil@kovalev.website>
Date: Thu, 30 Nov 2023 19:18:21 +0300
Subject: [PATCH] Fix packaging when `which python` is `/sbin`

On many systems, `/sbin` is a symbolic link to `/usr/bin`. When passing
`-p /usr` before `-P /usr/bin/python`, we set `BINDIR` env variable to
`/sbin` and then do not override it with `/usr/bin`. So, the package
archive contains `/sbin/cropgui` instead of `/usr/bin/cropgui`. It results in
the following issues during package installation:

  error: failed to commit transaction (conflicting files)
  cropgui: /sbin exists in filesystem (owned by filesystem)

Changing `-p` and `-P` options order resolves the issue.
---
 PKGBUILD | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/PKGBUILD b/PKGBUILD
index b4235ad..4c6597e 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -2,6 +2,7 @@
 # Contributor: eniac
 # Contributor: sabitmaulanaa
 # Contributor: tdehaeze
+# Contributor: Daniil Kovalev

 pkgname=cropgui
 arch=('any')
@@ -17,5 +18,5 @@ md5sums=('af229e7a38afbbb0a560ed9492bf978f')

 package() {
   cd $srcdir/$pkgname-$pkgver
-  ./install.sh -f gtk -t $pkgdir -p /usr -P /usr/bin/python
+  ./install.sh -f gtk -t $pkgdir -P /usr/bin/python -p /usr
 }
--
2.43.0

oriba commented on 2023-07-11 12:07 (UTC)

@jose1711: thank you for the package-update. It works well! :-)

jose1711 commented on 2023-06-28 21:11 (UTC)

thank you all for your suggestions, please see the updated PKGBUILD and feel free to review & comment.

oriba commented on 2023-06-28 11:23 (UTC)

@gabor_zoka: I wonder why the last tag of cropgui is from 2021. With a new tag, there is a new chance to have this AUR püackage updated - and other problems resolved also here. Regarding AUR rules, flagging as out-of-date is not to be used if a package is broken (which this one is). So maybe triggering the cropgui project to add a new tag would make it easier to get updates here.

oriba commented on 2023-06-15 02:30 (UTC)

The program does not work, when using this package:

Traceback (most recent call last):
  File "/usr/bin/cropgui", line 18, in <module>
    from cropgui_common import *
ModuleNotFoundError: No module named 'cropgui_common'

tdehaeze commented on 2023-06-01 08:43 (UTC)

Hi Jose1711,

A very nice pull request has been merged. Would it be possible to update the package to take into account this merge request (https://github.com/jepler/cropgui/pull/104).

Thanks!

gabor_zoka commented on 2023-04-07 13:10 (UTC) (edited on 2023-04-07 13:14 (UTC) by gabor_zoka)

Hello jose1711,

Would you consider this patch, please? It would make the source more recognizable, and less chance for name collision.

Thank you,

Gabor

--- a/PKGBUILD
+++ b/PKGBUILD
@@ -8,7 +8,7 @@ pkgdesc="a simple tool for lossless jpeg cropping"
 license=('GPL')
 url="https://github.com/jepler/cropgui"
 depends=('libjpeg' 'python-pillow' 'imagemagick' 'python-gobject' 'gtk3' 'perl-image-exiftool')
-source=("https://github.com/jepler/cropgui/archive/v${pkgver}.tar.gz")
+source=("${pkgname}-${pkgver}.tar.gz::https://github.com/jepler/cropgui/archive/v${pkgver}.tar.gz")
 md5sums=('337ba42c302793620b68085f88a28498')

 package() {

sabitmaulanaa commented on 2022-12-16 07:58 (UTC) (edited on 2022-12-16 07:58 (UTC) by sabitmaulanaa)

This is the patch that @eniac refer to.

--- cropgtk_orig.py 2021-10-04 20:33:02.398149075 +0200
+++ cropgtk.py  2021-10-04 20:34:04.880647475 +0200
@@ -220,7 +220,7 @@
         return self.result

 display = gdk.Display().get_default()
-wa = display.get_primary_monitor().get_workarea()
+wa = display.get_monitor(0).get_workarea()
 max_h = wa.height - 192
 max_w = wa.width - 64

eniac commented on 2022-02-04 12:41 (UTC)

I was able to fix the AttributeError issue on Wayland by applying the patch at https://pythonissues.com/issues/2146027

eniac commented on 2021-12-21 18:08 (UTC) (edited on 2021-12-21 18:08 (UTC) by eniac)

Tried both cropgui-git and cropgui, I always get the following error, any idea?

Stepconf encountered an error
Traceback:
File "/usr/bin/cropgui", line 223, in <module>
wa = display.get_primary_monitor().get_workarea()
AttributeError: 'NoneType' object has no attribute 'get_workarea'