Bug 294641 - Review Request: aboot - A bootloader which can be started from the SRM console.
Summary: Review Request: aboot - A bootloader which can be started from the SRM console.
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: alpha
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-09-18 12:54 UTC by Oliver Falk
Modified: 2007-11-30 22:12 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-10-05 08:31:24 UTC
Type: ---
Embargoed:
j: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Oliver Falk 2007-09-18 12:54:49 UTC
Spec URL: http://buildsys.zero42.at/mnt/koji/packages/aboot/1.0_pre20040408/1.fc8/src/aboot.spec
SRPM URL: http://buildsys.zero42.at/mnt/koji/packages/aboot/1.0_pre20040408/1.fc8/src/aboot-1.0_pre20040408-1.fc8.src.rpm
Description:
The aboot program is the preferred way of booting Linux when using SRM
firmware (the firmware normally used to boot an DEC UNIX). Aboot supports
the creation of bootable block devices and contains a program which can
load Linux kernels from a filesystem which is bootable by SRM.  Aboot
also supports direct booting from various filesystems (ext2, ISO9660,
UFS), booting of executable object files (ELF and ECOFF), booting of
compressed kernels, network booting (using bootp), partition tables in
DEC UNIX format, and interactive booting and default configurations for
SRM consoles that cannot pass long option strings.

If you are installing Fedora or Red Hat Linux on an Alpha, you'll need to
install the aboot package.

Comment 1 Jason Tibbitts 2007-09-20 21:56:28 UTC
It's going to be tough for many people to review this since we don't have Alphas
so ExclusiveArch will stop us.  I actually have Alphas (Some PW500s and a DS20)
but I'm not up to digging them out of storage at the moment.

Any chance you could provide links to a built package and the build logs?  And
maybe some rpmlint output as well?

Comment 2 Oliver Falk 2007-09-21 07:26:31 UTC
I've checked rpmlint. Yes, there where some small warnings that where fixed now.

My only problem is:
[oliver@gosa SPECS]$ rpmbuild -bs aboot.spec --nodeps
Wrote: /home/oliver/rpmbuild/SRPMS/aboot-1.0_pre20040408-2.fc8.src.rpm
[oliver@gosa SPECS]$ rpmlint
/home/oliver/rpmbuild/SRPMS/aboot-1.0_pre20040408-2.fc8.src.rpm
aboot.src: W: invalid-license GPL

I don't know how to handle this. The source doesn't state the exact version and
SF project page also states GPL. Is there some Wiki for such issues?


I've locally built aboot pkg on my Alpha and then imported; However, after I
have fixed the rpmlint warnings now, I've submitted a new build to alpha koji.
Please stand by, I will make a note within this bug as soon as the build is done.

Comment 3 Oliver Falk 2007-09-21 07:28:40 UTC
and jason, if you find some time, get your alphas out, update 'em to latest and
greatest devel tree and put it online for our alpha koji :-P

Comment 4 Mamoru TASAKA 2007-09-21 08:20:54 UTC
(In reply to comment #2)

> [oliver@gosa SPECS]$ rpmlint
> /home/oliver/rpmbuild/SRPMS/aboot-1.0_pre20040408-2.fc8.src.rpm
> aboot.src: W: invalid-license GPL
> 
> I don't know how to handle this. The source doesn't state the exact version and
> SF project page also states GPL. Is there some Wiki for such issues?

  License policy is changed and you have to specify the version
  for GPL/LGPL. Please refer to:
  http://fedoraproject.org/wiki/Licensing
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines

Comment 5 Mamoru TASAKA 2007-09-21 08:30:19 UTC
From my quick check license seems GPLv2+.

Comment 6 Oliver Falk 2007-09-21 08:34:53 UTC
Without having read both wikis... How did you find out?

I have changed the spec to GPLv2+.

Comment 7 Mamoru TASAKA 2007-09-21 10:30:19 UTC
(In reply to comment #6)
> Without having read both wikis... How did you find out?
  The source tarball contains GPLv2 text file and GPL license
  text says:
------------------------------------------------------
Each version is given a distinguishing version number.  If the Program
specifies a version number of this License which applies to it and "any
later version", you have the option of following the terms and conditions
either of that version or of any later version published by the Free
Software Foundation.  If the Program does not specify a version number of
this License, you may choose any version ever published by the Free Software
Foundation.
------------------------------------------------------
   From (quick) check of the source codes, some codes says
   "GPLv2 and any later", other says nothing. In this case
   license is GPLv2+.





Comment 8 Jason Tibbitts 2007-09-21 18:21:25 UTC
Any chance you'd have a link to the build package and build logs I could take a
look at?

I actually have one Alhpa machine handy (a Compaq XP1000), but I've never
installed Linux on it and have no idea how to begin, assuming that it's not too
ancient to actually run anything useful.

Comment 9 Oliver Falk 2007-09-22 06:27:10 UTC
Buildlogs will be here: http://buildsys.zero42.at/koji/taskinfo?taskID=42104, as
soon as finished....

XP1000 is fine, not the fastest one, but for testing well enough. Installing
Fedora Linux on that machine, I would wait for our new isos that I will
hopefully produce soon... It will include new kernel and new glibc - some
syscall fixes that I currently don't want to ship via yum, as it might break
(many) things - especially if you don't install it the RightWay(tm). :-)

Comment 10 Oliver Falk 2007-09-22 20:32:04 UTC
Build is done. You can now have a look at it....

Comment 11 Oliver Falk 2007-09-24 10:06:06 UTC
just *bing* :-)

Comment 12 Oliver Falk 2007-09-26 07:17:24 UTC
Duh? Why 4Suite!?

Comment 13 Jason Tibbitts 2007-09-26 15:28:27 UTC
Sometimes the component gets changed randomly.  I don't know if this is a
firefox bug or a bugzilla bug.  It's no big deal; I just change it back when I
have something else to add.

Comment 14 Jason Tibbitts 2007-09-26 17:14:46 UTC
A few complaints:

the manpages are executable, which rpmlint dutifully complains about:
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/e2writeboot.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/sdisklabel.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man1/isomarkboot.1.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/swriteboot.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/abootconf.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man1/netabootwrap.1.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man5/aboot.conf.5.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/aboot.8.gz

I notice that the normal set of compiler flags aren't used.  Now, this is a
bootloader so I can understand why, although there are userspace programs
included which perhaps should be built like any other userspace program.  Given
that this is for Alpha, though, I can't even be truly sure what the proper
compilation flags are.

This package does not meet the versioning guidelines; when 1.0 is released, it
will sort lower than the current package name.  The guidelines specify the
proper version and release to be used as:
  1.0-0.2.pre20040408
You can increment the '2' for each new revision, and when 1.0 is released you
can just use "1.0-1" without worrying about any sorting issues.

There's a COPYING file in the tarball, which must be included in the package.

* source files match upstream:
   a8ae8f2bf549c1cc79ea66a0a11c8db5c0257ce0d94b97418eb1c658723b12d2  
   aboot-1.0_pre20040408.tar.bz2
X package does not meet versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
X license text included in tarball but not in package.
* latest version is being packaged.
* BuildRequires are proper.
? compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, alpha).
* debuginfo package looks complete.
X rpmlint has valid complaints.
* final provides and requires are sane.
* %check is not present; no test suite upstream.  I have no way to test this 
  pacakge.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
X file permissions are not appropriate (executable manpages)
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

Comment 15 Oliver Falk 2007-09-27 16:27:02 UTC
I think I have fixed everything now:
http://buildsys.zero42.at/koji/buildinfo?buildID=11075

[oliver@gosa alpha]$ rpmlint aboot-1.0-0.1.pre20040408.fc8.alpha.rpm 
[oliver@gosa alpha]$ 


Please note. I've added a patch to include the rpm optflags... I don't know if
the package is *working*. But next time I boot my as1000a i will install updated
aboot and try... I don't think that the optflags will break anything.

Comment 16 Jason Tibbitts 2007-09-27 18:29:41 UTC
OK, cool. rpmlint shuts up, the COPYING file is in there, the manpages aren't
executable, and the version looks good.

Of course you know I can't test this so I'll leave that to you; if the compiler
flag change breaks things then I have no issues with you reverting it.

APPROVED

Comment 17 Oliver Falk 2007-09-28 07:48:41 UTC
Thx for the review. It was a pleasure for me.

Well, yes if the compiler flags break something I have to revert it, but I don't
think they will.

Comment 18 Oliver Falk 2007-09-28 07:56:57 UTC
New Package CVS Request
=======================
Package Name: aboot
Short Description: A bootloader which can be started from the SRM console
Owners: oliver
Branches: devel
InitialCC: oliver
Cvsextras Commits: yes

Comment 19 Kevin Fenzi 2007-09-28 21:31:54 UTC
cvs done.


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