Package Details: octopi 0.17.0-1

Git Clone URL: https://aur.archlinux.org/octopi.git (read-only, click to copy)
Package Base: octopi
Description: A powerful Pacman frontend using Qt libs
Upstream URL: https://github.com/aarnt/octopi
Licenses: GPL-2.0-or-later
Conflicts: alpm_octopi_utils
Submitter: ImNtReal
Maintainer: xiota
Last Packager: xiota
Votes: 1637
Popularity: 34.44
First Submitted: 2013-09-03 23:42 (UTC)
Last Updated: 2025-02-18 19:38 (UTC)

Dependencies (21)

Required by (0)

Sources (2)

Latest Comments

« First ‹ Previous 1 2 3 4 5 6 7 8 9 10 11 12 .. 40 Next › Last »

xiota commented on 2024-03-18 23:26 (UTC)

@matmoul The problem is you are not following general, wide-spread best practices, namely, saving source files with version and extension.

The format for entries in the source array is: filename::url

The folder that is extracted depends on the contents of the archive. Upstream could change it and force you to rewrite the PKGBUILD.

I usually don't like split packages, but in this case, it is far better than separate packages that attempt to use (or redownload) the same source and depend on each other.

I suggest adding yochananmarqos as comaintainer to the associated packages so he can fix them. He needs to be added to all of them so he can rename the conflicting packages while deletion is pending. (I would also be able to, but yochananmarqos has already created an alternate PKGBUILD.)

matmoul commented on 2024-03-18 23:06 (UTC) (edited on 2024-03-18 23:07 (UTC) by matmoul)

@xiota I've really tested the line and it work and the line from @yochananmarqos to ...

It seems I haven't figured out all of the source prefix yet.
I thought this would impact the cd in the functions.

But that doesn't explain to me why you're having this problem.
I used tryzen for a long time before using my internal solution without ever encountering a problem.

yochananmarqos commented on 2024-03-18 22:28 (UTC)

...and even better, how about an improved split PKGBUILD? (paste expires in one year)

yochananmarqos commented on 2024-03-18 22:24 (UTC)

...or, y'know... KISS and don't use unnecessary variables:

source=("${pkgname}-${pkgver}.tar.gz::https://github.com/aarnt/octopi/archive/v${pkgver}.tar.gz")

xiota commented on 2024-03-18 22:23 (UTC) (edited on 2024-03-18 22:27 (UTC) by xiota)

Arch Wiki PKGBUILD#Sources

Warning: The downloaded source filename must be unique because the SRCDEST directory can be the same for all packages. For instance, using the version number of the project as a filename potentially conflicts with other projects with the same version number. In this case, the alternative unique filename to be used is provided with the syntax source=('unique_package_name::file_uri'); e.g. source=("$pkgname-$pkgver.tar.gz::https://github.com/coder/program/archive/v$pkgver.tar.gz").

Changing that single line does not break the package because the archive is unpacked and its contents are used for the rest of the build process.

matmoul commented on 2024-03-18 22:16 (UTC)

@xiota As I build my packages in a clean chroot, I would be very curious to know how you manage your packages and why you encountered this problem?

The temporary name are only used for building the package.

Can you tell me in which folder the conflict is occurring for you?

I don't understand why your cache is not updating, because with all octopi users, I have never seen this request.

The last point to clarify "Violates Arch package guidelines", a link?

PS: By changing the indicated code line as is, it breaks the package...

yochananmarqos commented on 2024-03-18 21:59 (UTC) (edited on 2024-03-18 22:00 (UTC) by yochananmarqos)

@xiota: I wasn't aware of anything and neither was anyone else here--hence why @matmoul had to ask. I only commented in a light-hearted manner to encourage you to explain. You did, thank you.

xiota commented on 2024-03-18 21:58 (UTC)

@matmoul The following packages are also affected: dokytree, g810-led, octopi-notifier-frameworks, octopi-notifier-qt5, plasma-addons-customdesktopmenu, walinuxagent, x48ng.

@yachananmarqos You are fully aware of the defect and reasons it needs to be corrected. If you have nothing constructive to contribute, you shouldn't bother commenting.

yochananmarqos commented on 2024-03-18 21:49 (UTC)

@matmoul: Sorry, @xiota only makes demands with no explanation. :P

@xiota: Please prove me wrong. :D

xiota commented on 2024-03-18 21:49 (UTC) (edited on 2024-03-18 21:56 (UTC) by xiota)

Currently, the file is saved without version or extension as octopi. This has several problems:

  • More difficult to identify the nature of the file without version or extension.
  • File is cached, and not redownloaded, causing checksum failure, requiring manual deletion by the user.
  • Interferes with aur/octopi-git when $SRCDEST is set.
  • Violates Arch package guidelines.

« First ‹ Previous 1 2 3 4 5 6 7 8 9 10 11 12 .. 40 Next › Last »