On Tue, 12 Jul 2016 12:01:22 -0400 Storm Dragon via aur-general wrote:

Howdy, I am working on a PKGBUILD for a new GNU Social client called GnuSocialShell. I made one for the git version, because this is being rapidly developed, and I don't believe there is an actual release yet anyway. The problem is, it haults with an error when trying to copy files in place, and I'm not sure why. I have gone over this file and can't find anything wrong with it. It's probably something simple I'm just overlooking though. Thanks for any help :) Storm

Looks like a very badly written Makefile. You'll have to install the files to $pkgdir yourself.

On 07/12/2016 12:11 PM, Doug Newgard wrote:

On Tue, 12 Jul 2016 12:01:22 -0400 Storm Dragon via aur-general wrote:

...

Howdy, I am working on a PKGBUILD for a new GNU Social client called GnuSocialShell. I made one for the git version, because this is being rapidly developed, and I don't believe there is an actual release yet anyway. The problem is, it haults with an error when trying to copy files in place, and I'm not sure why. I have gone over this file and can't find anything wrong with it. It's probably something simple I'm just overlooking though. Thanks for any help :) Storm

Looks like a very badly written Makefile. You'll have to install the files to $pkgdir yourself.

That, or write a competently-written Makefile *from scratch* and contribute it upstream if you want to help the project. Seriously. That Makefile looks like someone decided "Makefiles are traditional, so we need one", but had no real idea how to make one, so they just created aliases for the commands they usually used to manually compile and install it. -- Eli Schwartz

2016-07-12 19:55 GMT-03:00 Rafael Fontenelle :

2016-07-12 13:01 GMT-03:00 Storm Dragon via aur-general < aur-general@archlinux.org>:

...

Howdy, I am working on a PKGBUILD for a new GNU Social client called GnuSocialShell. I made one for the git version, because this is being rapidly developed, and I don't believe there is an actual release yet anyway. The problem is, it haults with an error when trying to copy files in place, and I'm not sure why. I have gone over this file and can't find anything wrong with it. It's probably something simple I'm just overlooking though. Thanks for any help :) Storm

Hello there.

My comments on this situation and your PKGBUILD:

1st - The PKGBUILD has CRLF. You need to format it to Unix (use 'dos2unix' command)

2nd - Your command line to get version doesn't seem to be correct. 'Main.c' reports current version 0.8, just like 'git shortlog' history shows. But you've got '1' from what doesn't look to be a version. Please review that.

3rd - Not the main issue, but remember to set arch (don't use 'any')

4th - Makefile is copying files to directly to '/etc' and '/usr', instead of allowing fakeroot to work with a variable like $(DESTDIR) before that path. That's an upstream bug, that can be fixed with the attached patch

5th - Makefile tries to copy to directories without making sure they exist. Again, upstream bug fixed with my patch.

My suggestion is to fix yourself the first 3 issues and file a bug report for the last 2.

Best regards, Rafael Fontenelle

Now with attachment. Rafael Fontenelle

On Tue, 12 Jul 2016 20:05:43 -0300, Rafael Fontenelle wrote:

Now with attachment.

Third time lucky.

Howdy, I have no clue how it got set to crlf. I made this in Vim on Arch lol. I guess the original was from a template with the wrong format. This is fixed now, however. I also think I have a working fix for the version issue. This really does compile on any architecture, including ARMv7 etc. I applied your patch and did a pull request, thanks so much for providing that. Also, thank you for the tips. Storm On Tue, Jul 12, 2016 at 08:05:43PM -0300, Rafael Fontenelle wrote:

2016-07-12 19:55 GMT-03:00 Rafael Fontenelle :

...

2016-07-12 13:01 GMT-03:00 Storm Dragon via aur-general < aur-general@archlinux.org>:

...

Howdy, I am working on a PKGBUILD for a new GNU Social client called GnuSocialShell. I made one for the git version, because this is being rapidly developed, and I don't believe there is an actual release yet anyway. The problem is, it haults with an error when trying to copy files in place, and I'm not sure why. I have gone over this file and can't find anything wrong with it. It's probably something simple I'm just overlooking though. Thanks for any help :) Storm

Hello there.

My comments on this situation and your PKGBUILD:

1st - The PKGBUILD has CRLF. You need to format it to Unix (use 'dos2unix' command)

2nd - Your command line to get version doesn't seem to be correct. 'Main.c' reports current version 0.8, just like 'git shortlog' history shows. But you've got '1' from what doesn't look to be a version. Please review that.

3rd - Not the main issue, but remember to set arch (don't use 'any')

4th - Makefile is copying files to directly to '/etc' and '/usr', instead of allowing fakeroot to work with a variable like $(DESTDIR) before that path. That's an upstream bug, that can be fixed with the attached patch

5th - Makefile tries to copy to directories without making sure they exist. Again, upstream bug fixed with my patch.

My suggestion is to fix yourself the first 3 issues and file a bug report for the last 2.

Best regards, Rafael Fontenelle

Now with attachment.

Rafael Fontenelle

-- Powered by Arch Linux! I am registered Linux user number 508465: https://linuxcounter.net/user/508465.html My blog, Thoughts of a Dragon: http://www.stormdragon.tk/ get my public PGP key: gpg --keyserver wwwkeys.pgp.net --recv-key 43DDC193 Twitter and Facebook are so ... yesteryear. Get your 2MB Social account TODAY! http://2mb.social/main/register Need a safe and easy way to backup and share files? Try Dropbox: http://db.tt/jeY50HR "No life anymore only darkness and pain, shape of shadow no flesh nor blood. All passions now gone no reason to be, cursed by the Lord of the Rings." Battlelore - The Curse of the Kings

On 07/13/2016 12:30 PM, Storm Dragon via aur-general wrote:

Howdy, I have no clue how it got set to crlf. I made this in Vim on Arch lol. I guess the original was from a template with the wrong format. This is fixed now, however. I also think I have a working fix for the version issue.

The proper fix for the version issue is for upstream to tag their releases, and use the `git describe` example on the Arch Wiki.

This really does compile on any architecture, including ARMv7 etc.

Right, but that isn't what "arch" means in the context of a PKGBUILD. Maybe it is time to review the PKGBUILD definition? https://wiki.archlinux.org/index.php/PKGBUILD#arch

I applied your patch and did a pull request, thanks so much for providing that. Also, thank you for the tips. Storm

As mentioned several times, that Makefile is a wreck. The *proper* fix looks a lot more like this: http://pastebin.com/raw/EUN7P1e5 -- Eli Schwartz