Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

googleearth PKGBUILD parse problems on 32-bit Arch #174

Closed
hutou opened this Issue Dec 14, 2013 · 8 comments

Comments

Projects
None yet
4 participants

hutou commented Dec 14, 2013

Trying to install GoogleEarth on my 32bits Arch system with

$ sudo aura -A google-earth

gives me the following error

aura >>= Determining dependencies...
aura >>= Dependency checking failed for these reasons:
The dependency `lib32-fontconfig` could not be found. You may need to search for a package to satisfy it.

aura or PKGBUILD culprit ?

Package googleearth correctly installed with

sudo aura -A ld-lsb
yaourt -S google-earth

Separate install of ld-lsb was needed because, on first try, "yaourt -S google-earth" complained about lib32-glibc, dependency of ld-lsb

See comments on https://aur.archlinux.org/packages/google-earth/

Owner

fosskers commented Dec 14, 2013

That is quite a complicated PKGBUILD and I imagine the Aura's Bash parser is failing to parse that if block correctly.

Contributor

eazar001 commented Dec 14, 2013

@hutou I inspected lib32-fontconfig a while ago and noticed that those files modules were built for 64-bit architecture. I think editing the dependency to fontconfig is more appropriate for your system. However the behavior with aura itself still remains unresolved, as people do have different ISAs. Not sure if that scales correctly, but my google-earth seems to be running fine.

Contributor

eazar001 commented Dec 14, 2013

p.s., yes I know it says (32-bit), but the architecture indicator suggests different.

hutou commented Dec 15, 2013

It seems aura fails to parse correctly the "old"

[ $foo = "bar" ]

or

[ $foo = "bar" ] || [ $bar != 1 ] foo="barbar"

Bash constructs !
(see thread above)

Owner

fosskers commented Dec 16, 2013

I believe it only tests for equality at the moment, not inequality.

Contributor

joneshf commented Dec 16, 2013

I'd like to help out with aura, and this seems like an easy enough task.

So looking at the bash parser, it seems we'd have to modify these areas in order to support additional operations:

Add neq parse here: https://github.com/fosskers/aura/blob/master/Bash/Parser.hs#L173-L182
Add another type to the sum here: https://github.com/fosskers/aura/blob/42fc7625fe9f0930f078c7b1d02f7ce05f1af93f/Bash/Base.hs#L40
Add a check for neq here: https://github.com/fosskers/aura/blob/42fc7625fe9f0930f078c7b1d02f7ce05f1af93f/Bash/Simplify.hs#L91-L100
Add some replacement logic here:
https://github.com/fosskers/aura/blob/42fc7625fe9f0930f078c7b1d02f7ce05f1af93f/Bash/Simplify.hs#L61-L64

Am I missing anything?

@joneshf joneshf added a commit to joneshf/aura that referenced this issue Dec 16, 2013

@joneshf joneshf Add support for not equal in Bash parser. Re: #174. 14b9842
Owner

fosskers commented May 22, 2014

This will be fixed when AUR 3.0 is launched, as Aura won't have to parse Bash anymore.

@fosskers fosskers referenced this issue Nov 20, 2015

Open

Aura 1.4 #377

5 of 24 tasks complete
Owner

fosskers commented Nov 20, 2015

Fixed.

@fosskers fosskers closed this Nov 20, 2015

@fosskers fosskers modified the milestone: Aura 1.4 Feb 14, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment