Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 542560 - games-engines/scummvm-tools-1.7.0: No files installed
Summary: games-engines/scummvm-tools-1.7.0: No files installed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2015-03-08 13:39 UTC by Kris Scott
Modified: 2015-03-09 23:15 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Fixed ebuild (scummvm-tools-1.7.0-r1.ebuild,1.34 KB, text/plain)
2015-03-08 14:00 UTC, Kris Scott
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kris Scott 2015-03-08 13:39:43 UTC
The ebuild for this package installs documentation only, no binaries. 	

There is a line in the ebuild which says:

for f in $(find . -type f -perm +1 -print); do

This does not work as -perm +1 is not a valid option for find. The ebuild should instead install each binary manaully based on use flags.

Reproducible: Always

Steps to Reproduce:
1. emerge scummvm-tools
2. equery f scummvm-tools

Actual Results:  
Only documentation installed.

Expected Results:  
Compiled binaries should be installed.
Comment 1 Kris Scott 2015-03-08 14:00:44 UTC
Created attachment 398392 [details]
Fixed ebuild

Updated the ebuild, removing the strange way of installing files. I am not sure if it was some holdover that just kept getting version bumped, or if there was a specific reasoning behind it.

I just replaced the src_install() with a barebones version that kept game permissions.
Comment 2 Andrew Savchenko gentoo-dev 2015-03-09 19:00:02 UTC
Please provide a patch instead of new ebuild.
Comment 3 Mr. Bones. (RETIRED) gentoo-dev 2015-03-09 21:54:40 UTC
-perm +1 is completely valid find syntax but the install was pretty messy with the latest version of scummvm-tools so I updated it to result in a nicer install.
Comment 4 Kris Scott 2015-03-09 23:15:09 UTC
I'd like to point out that -perm +1 is NOT valid. According to the find man page:

-perm +mode
This is no longer supported (and has been deprecated since 2005).  Use -perm /mode instead.

In the findutils on my system version 4.5.14-r1, trying to use "-perm +1" returns an error.