Bug 750898

Summary: Review Request: icaro - A robot automation language
Product: [Fedora] Fedora Reporter: Larry Letelier <geek>
Component: Package ReviewAssignee: Peter Lemenkov <lemenkov>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: echevemaster, geek, guillermo.gomez, lemenkov, notting, package-review, rc040203
Target Milestone: ---Flags: lemenkov: fedora-review?
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-20 02:30:44 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
adds missing include header none

Description Larry Letelier 2011-11-02 18:12:49 UTC
Spec URL: http://lletelier.fedorapeople.org/icaro/icaro.spec
SRPM URL: http://lletelier.fedorapeople.org/icaro/icaro-0.6.2-1.290311.fc15.src.rpm
Description: Icaro is a software developed with the intention of bringing,
in a manner transparent to the user the basics of programming 
(iterations, repetition, recursion, if conditional) of a 
physical context such as a robot or automation system.

Comment 2 Peter Lemenkov 2012-03-02 17:45:55 UTC
I'll review it (and will sponsor you).

Comment 3 Larry Letelier 2012-03-02 17:54:25 UTC
Thanks Peter,

-- LL

Comment 4 Peter Lemenkov 2012-03-10 12:16:30 UTC
Created attachment 569080 [details]
adds missing include header

Comment 5 Peter Lemenkov 2012-03-10 12:16:46 UTC
Few notes.

* The package fails to build on non-x86 arches due to lack of <sys/io.h>. Please add "ExclusiveArch:  %{ix86} x86_64" for now. I think it's a good idea to add check for this and for some other required files to the configure.ac but that's not related to the packaging and won't block the review process - that's just my friendly advice.

* The package fails to build due to the missing include file - <unistd.h>. Please see patch attached.

http://koji.fedoraproject.org/koji/taskinfo?taskID=3877160

Btw what's your FAS name?

Comment 6 Guillermo Gómez 2012-03-13 22:00:49 UTC
(In reply to comment #2)
> I'll review it (and will sponsor you).

AFAIK LL is already sponsored, he just need to be reviewed.

Comment 7 Ralf Corsepius 2012-03-14 07:02:42 UTC
2 nits:

* %release is poorly chosen. It needs to contain a fedora specific release number.
You are trying to reuse some upstream release number, which doesn't leave the fedora package maintainer an option to increment it.

* Please move the "chmod -x" from %build into %prep.

Comment 8 Eduardo Echeverria 2013-02-20 02:30:44 UTC

*** This bug has been marked as a duplicate of bug 912930 ***