Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 181831 - Review Request: bitbake - Build Tool
Summary: Review Request: bitbake - Build Tool
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: John Mahowald
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-02-16 23:05 UTC by Andreas Thienemann
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-04 03:17:04 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Andreas Thienemann 2006-02-16 23:05:06 UTC 
Spec Name or http://helena.bawue.de/~ixs/bitbake/bitbake.spec
SRPM Name or Url: http://helena.bawue.de/~ixs/bitbake/bitbake-1.3.3-1.src.rpm
Description:
BitBake is a simple tool for the execution of tasks. It is derived from
Portage, which is the package management system used by the Gentoo Linux
distribution. It is most commonly used to build packages, and is used as the
basis of the OpenEmbedded project.
Comment 1 Andreas Thienemann 2006-02-16 23:11:26 UTC 
JFTR: rpmlint complains about some .py scripts not being set to executable. This
makes sense, as not all of these scripts have the #!/usr/bin/python header included.
Comment 2 Michael J Knox 2006-03-07 21:38:59 UTC 
builds cleanly on FC4. rpmlint shows the issues you have already stated and no
others. Bitbake does recommend pysco, but pysco is not in FCE, perhaps that
should get pulled in at some stage, but is not a show stopper. 

Looks good.
Comment 3 Andreas Thienemann 2006-03-20 14:54:44 UTC 
(In reply to comment #2)
> Bitbake does recommend pysco, but pysco is not in FCE, perhaps that
> should get pulled in at some stage, but is not a show stopper. 
psyco is purely optional as it would speed up a bitbake run by optimizing the
python code.
Furthermore psyco runs only on i386 architectures.

> Looks good. 
Thx. If it's okay with you, I'd upload the package into cvs then.
Comment 4 John Mahowald 2006-04-08 20:05:41 UTC 
Do not use /usr/%{_lib} when specifying python's site-packages.

Instead, install fedora-rpmdevtools and use fedora-newrpmspec --type python. 
This template has a %python_sitelib macro for this, which evaluates to /usr/lib
on x86_64 for architecture independent packages like this one.
Comment 5 Andreas Thienemann 2006-04-13 03:26:53 UTC 
Thx for the advice.

.Spec is cleaned up and at http://home.bawue.de/~ixs/bitbake/
Comment 6 John Mahowald 2006-05-01 16:22:38 UTC 
- rpmlint checks return:
E: bitbake non-executable-script /usr/lib/python2.4/site-packages/bb/build.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/parse/parse_py/BBHandler.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/local.py 0644
E: bitbake non-executable-script /usr/lib/python2.4/site-packages/bb/data.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/cvs.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/svn.py 0644
E: bitbake non-executable-script /usr/lib/python2.4/site-packages/bb/event.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/wget.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/parse/parse_py/ConfHandler.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/__init__.py 0644
E: bitbake non-executable-script /usr/lib/python2.4/site-packages/bb/cache.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/parse/parse_py/__init__.py 0644
E: bitbake non-executable-script /usr/lib/python2.4/site-packages/bb/__init__.py
0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/git.py 0644
E: bitbake non-executable-script /usr/lib/python2.4/site-packages/bb/shell.py 0644
E: bitbake non-executable-script
/usr/lib/python2.4/site-packages/bb/fetch/svk.py 0644

Which are ignorable.


- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime

Building html, xhtml, text, and pdf docs seems a bit redundant. You can drop
html if you like.

- no need for .desktop file 

APPROVED
Comment 7 Andreas Thienemann 2006-05-04 03:17:04 UTC 
Thx for the review. -> NEXTRELEASE
Note You need to log in before you can comment on or make changes to this bug.