Bug 225246 - Merge Review: anaconda
Merge Review: anaconda
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jason Tibbitts
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-29 16:00 EST by Nobody's working on this, feel free to take it
Modified: 2012-11-29 13:43 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2009-01-13 16:05:25 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tibbs: fedora‑review+


Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-29 16:00:47 EST
Fedora Merge Review: anaconda

http://cvs.fedora.redhat.com/viewcvs/devel/anaconda/
Comment 1 Jason Tibbitts 2008-12-19 13:23:56 EST
Here are some comments copied from my mailing list post:

No %{?dist} tag, but I understand why you folks have no need of that.

Is the source available for download anywhere?  The Source0: URL
should be downloadable if at all possible.  The spec should at minimum
include some instructions for checking out of an SCM.

As of FC-5, it is not actually necessary to have:
  Requires(post): desktop-file-utils >= %{desktopfileutilsver}
  Requires(postun): desktop-file-utils >= %{desktopfileutilsver}
as long as you use the recommended update-desktop-database
scriptlets.
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#desktop-database

%post
/usr/bin/update-desktop-database %{_datadir}/applications &> /dev/null || :

%postun
/usr/bin/update-desktop-database %{_datadir}/applications &> /dev/null || :

Is there any reason to keep this bit:
%triggerun -- anaconda < 8.0-1
/sbin/chkconfig --del reconfig >/dev/null 2>&1 || :

Anaconda < 8.0 was, what, pre-RHL-8?

There's some minor permission weirdness, like
/usr/lib/anaconda-runtime/boot/syslinux.cfg and
/usr/lib/anaconda-runtime/pyrc.py are executable, and also
some shell scripts that don't have a shebang line like
/usr/lib/anaconda-runtime/mk-images.s390.  I'll spare everyone here
the full listing; just run rpmlint and look at the
script-without-shebang errors.
Comment 2 Hans de Goede 2009-01-06 03:10:09 EST
(In reply to comment #1)

Thanks! I've addressed all your comments in 11.5.0.4, which should be in the next rawhide push.

> Is the source available for download anywhere?  The Source0: URL
> should be downloadable if at all possible.  The spec should at minimum
> include some instructions for checking out of an SCM.
> 

Done

> As of FC-5, it is not actually necessary to have:
>   Requires(post): desktop-file-utils >= %{desktopfileutilsver}
>   Requires(postun): desktop-file-utils >= %{desktopfileutilsver}
> as long as you use the recommended update-desktop-database
> scriptlets.
> http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#desktop-database
> 
> %post
> /usr/bin/update-desktop-database %{_datadir}/applications &> /dev/null || :
> 
> %postun
> /usr/bin/update-desktop-database %{_datadir}/applications &> /dev/null || :
> 

Fixed.

> Is there any reason to keep this bit:
> %triggerun -- anaconda < 8.0-1
> /sbin/chkconfig --del reconfig >/dev/null 2>&1 || :
> 

Removed.

> Anaconda < 8.0 was, what, pre-RHL-8?
> 
> There's some minor permission weirdness, like
> /usr/lib/anaconda-runtime/boot/syslinux.cfg and
> /usr/lib/anaconda-runtime/pyrc.py are executable, and also
> some shell scripts that don't have a shebang line like
> /usr/lib/anaconda-runtime/mk-images.s390.  I'll spare everyone here
> the full listing; just run rpmlint and look at the
> script-without-shebang errors.

I've fixed most rpmlint nagging, remaining:

anaconda.src:205: E: hardcoded-library-path in %{_prefix}/lib/anaconda
anaconda.src:206: E: hardcoded-library-path in %{_prefix}/lib/anaconda-runtime
* python code, so no need for lib64

anaconda.x86_64: W: file-not-utf8 /usr/share/doc/anaconda-11.5.0.3/ChangeLog
* automatically generated from git commit messages later ones are all UTF-8,
  cannot use iconv, as newer parts are already UTF-8, cannot manually fix as
  generated on the fly.

anaconda.x86_64: E: explicit-lib-dependency libbdevid-python
anaconda.x86_64: E: explicit-lib-dependency libuser-python
anaconda.x86_64: E: explicit-lib-dependency libxml2-python

These are actually all python bindings, and this are not explicit-lib-dependencies.
Comment 3 Jason Tibbitts 2009-01-13 16:05:25 EST
Yes, the explicit lib dependency bits are obviously bogus and you can't really fix the changelog.  The hardcoded-library-path complaints are bogus as well.  That clears rpmlint.

The other issues have all been fixed, so I'd say we're done.  Thanks for your attention.

APPROVED
Comment 4 Brian Lane 2012-11-29 12:37:12 EST
Package Change Request
======================
Package Name: anaconda
New Branches: f18-beta
Owners: bcl clumens
InitialCC: 

We need a branch in order to do a new beta build for secondary arches.
Comment 5 Jon Ciesla 2012-11-29 12:45:45 EST
Is a branch needed, or a buildroot tag in koji?
Comment 6 Brian Lane 2012-11-29 13:43:07 EST
Just a branch, thanks.

Note You need to log in before you can comment on or make changes to this bug.