Community Packages

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#40711 - [pkgbuild-introspection] mkaurball doesn't set $CARCH

Attached to Project: Community Packages
Opened by (Det) - Thursday, 05 June 2014, 15:04 GMT
Last edited by Dave Reisner (falconindy) - Wednesday, 06 August 2014, 17:38 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Lukas Fleischer (lfleischer)
Dave Reisner (falconindy)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No

Details

Currently, mkaurball doesn't understand $CARCH.

This means that a PKGBUILD that says:

_arch=i386
[ "$CARCH" = 'x86_64' ] && _arch=amd64
source=("$_arch.deb")

Produces the following on a 64-bit system in the .AURINFO:

source = i386.deb
This task depends upon

Closed by  Dave Reisner (falconindy)
Wednesday, 06 August 2014, 17:38 GMT
Reason for closing:  Won't fix
Additional comments about closing:  Needs fixing in makepkg to provide arch specific deps which AURINFO can then support.
Comment by Johannes Löthberg (demize) - Thursday, 05 June 2014, 15:53 GMT
Sourceballs aren't supposed to be architecture dependent, so that examples doesn't really make much sense.
Comment by Doug Newgard (Scimmia) - Thursday, 05 June 2014, 16:04 GMT
They are when you're downloading binaries.
Comment by (Det) - Thursday, 05 June 2014, 16:07 GMT
Right. Which is what the .deb stands for.

It's just more fool-proof, because if I do:

depends=('lib32-cairo')
[ "$CARCH" = 'i686' ] && depends=('cairo')
_arch=i386
[ "$CARCH" = 'x86_64' ] && _arch=amd64
source=("$_arch.deb")

I'm returned with:

depends = lib32-cairo
source = i386.deb
Comment by Dave Reisner (falconindy) - Thursday, 05 June 2014, 16:43 GMT
Enforcing CARCH just means the .AURINFO file will be wrong in some other way. If you want this to be fixed properly, work on support in makepkg for architecture specific attributes.
Comment by (Det) - Thursday, 05 June 2014, 16:53 GMT
Does that mean even after setting it, we wouldn't get a correct .AURINFO even for just the other arch?

Currently, we get a mix, which is a lot worse.
Comment by Johannes Löthberg (demize) - Thursday, 05 June 2014, 16:56 GMT
Simple fix: Just write your own .AURINFO.
Comment by (Det) - Thursday, 05 June 2014, 17:01 GMT
It's not a fix. It's just a workaround and it also means you need to edit the .AURINFO in the future by hand.
Comment by Johannes Löthberg (demize) - Thursday, 05 June 2014, 17:04 GMT
Not really, mkaurball is the workaround to having to write it manually, and seriously, is it that hard to change one or two lines?
Comment by (Det) - Thursday, 05 June 2014, 17:10 GMT
I'm not going to argue with you in drawing a line in what is lazy and not, but in the future you will be _required_ to use mkaurball to create packages to the AUR.

When that happens I'd much rather have them be as 'consistent' as possible, and not include i686 dependencies and x86_64 sources.

If we can prevent that, then I don't get what your problem is.
Comment by Johannes Löthberg (demize) - Thursday, 05 June 2014, 17:14 GMT
False, you don't need mkaurball at all, it's just a convenience wrapper.
Comment by Doug Newgard (Scimmia) - Thursday, 05 June 2014, 17:14 GMT
Det, the simple answer is to be consistent with which arch is conditional. Mixing them is the only time you'd get into trouble.
Comment by (Det) - Thursday, 05 June 2014, 17:18 GMT
@Kyrias, you can say that after writing a couple :).

@Scimmia, if that means the current implementation really is that hard to fix, then I guess I've got my answer. Thanks.
Comment by Johannes Löthberg (demize) - Thursday, 05 June 2014, 17:24 GMT
Det: And how do you know that I haven't already?
Comment by (Det) - Thursday, 05 June 2014, 17:26 GMT
Close it away, please :).

Loading...