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 181035 - Review Request: luks-tools
Summary: Review Request: luks-tools
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jochen Schmitt
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 166035
TreeView+ depends on / blocked
 
Reported: 2006-02-11 23:14 UTC by W. Michael Petullo
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-09-13 22:04:02 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Make luks-setup build clean on FC5 (deleted)
2006-06-08 17:25 UTC, David Hollis
no flags Details

Description W. Michael Petullo 2006-02-11 23:14:34 UTC
Spec Name or Url: http://www.flyn.org/SRPMS/luks-tools.spec
SRPM Name or Url: http://www.flyn.org/SRPMS/luks-tools-0.0.8-1.src.rpm
Description:
The luks-tools package contains various utilities for working with
LUKS-protected filesystems. HAL uses these utilities to automatically
mount encrypted volumes when they are attached to a system, provided
the user can produce the correct passphrase. These utilities are
written as separate programs to allow MAC systems like SELinux to
have fine-grained control over them.

Comment 1 Jochen Schmitt 2006-02-16 16:56:28 UTC
Local build failed:

checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking whether make sets $(MAKE)... (cached) yes
checking whether ln -s works... yes
checking for special C compiler options needed for large files... no
checking for _FILE_OFFSET_BITS value needed for large files... 64
checking for _LARGE_FILES value needed for large files... no
checking for pkg-config... /usr/bin/pkg-config
checking for GLIB - version >= 2.0.0... yes (version 2.7.4)
checking for cryptsetup... no
configure: error: cryptsetup executable not found in your path
Fehler: Bad exit status from /var/tmp/rpm-tmp.78258 (%build)


RPM build errors:
    InstallSourcePackage: Header V3 DSA signature: NOKEY, key ID cb278bd5
    user mike does not exist - using root
    group mike does not exist - using root
    user mike does not exist - using root
    group mike does not exist - using root
    Bad exit status from /var/tmp/rpm-tmp.78258 (%build)
                                     

Comment 2 W. Michael Petullo 2006-02-16 22:19:51 UTC
Spec Name or Url: http://www.flyn.org/SRPMS/luks-tools.spec
SRPM Name or Url: http://www.flyn.org/SRPMS/luks-tools-0.0.8-2.src.rpm

Comment 3 David Hollis 2006-06-08 17:25:16 UTC
Created attachment 130765 [details]
Make luks-setup build clean on FC5

The attached patch adds a BuildRequires on check-devel and adds /sbin:/usr/sbin
to the path to the call to %configure so that it picks up cryptsetup and other
tools that it uses.

Comment 4 W. Michael Petullo 2006-08-06 16:57:33 UTC
Spec Name or Url: http://www.flyn.org/SRPMS/luks-tools.spec
SRPM Name or Url: http://www.flyn.org/SRPMS/luks-tools-0.0.9-2.src.rpm

Comment 5 Kevin Fenzi 2006-09-02 20:26:44 UTC
Hey Jochen. You moved this to FE-REVIEW, but didn't assign it to yourself. 
I am doing that now. If you are not going to review this, go ahead and reassign 
to nobody and set it back to block FE-NEW. 

Comment 6 Jochen Schmitt 2006-09-04 16:34:22 UTC
I will got the following complaint:

Source luks-tools-0.0.9.tar.gz is different from upstream


Comment 7 W. Michael Petullo 2006-09-04 17:18:25 UTC
Luks-tools 0.0.10 is synchronized with the upstream version:

Spec Name or Url: http://www.flyn.org/SRPMS/luks-tools.spec
SRPM Name or Url: http://www.flyn.org/SRPMS/luks-tools-0.0.10-1.src.rpm

Comment 8 Jochen Schmitt 2006-09-07 16:06:59 UTC
Good:
+ Source match with upstream.
+ Mock build works fine.


Bad:
- Rpmlint complaints luks-tools
W: luks-tools non-executable-in-bin /usr/bin/gnome-luks-format.pyo 0644
W: luks-tools non-executable-in-bin /usr/bin/gnome-luks-format.pyc 0644


Comment 9 W. Michael Petullo 2006-09-07 22:05:00 UTC
Spec Name or Url: http://www.flyn.org/SRPMS/luks-tools.spec
SRPM Name or Url: http://www.flyn.org/SRPMS/luks-tools-0.0.10-2.src.rpm

Comment 10 Jochen Schmitt 2006-09-10 19:59:34 UTC
Bad:

- rpmlint luks-tools complaints:
$ rpmlint luks-tools-0.0.10-2.i386.rpm
W: luks-tools non-executable-in-bin /usr/bin/gnome-luks-format.pyo 0644
W: luks-tools non-executable-in-bin /usr/bin/gnome-luks-format.pyc 0644


Comment 11 W. Michael Petullo 2006-09-10 21:16:53 UTC
Okay, I was not familiar with brp-python-bytecompile.

Spec Name or Url: http://www.flyn.org/SRPMS/luks-tools.spec
SRPM Name or Url: http://www.flyn.org/SRPMS/luks-tools-0.0.11-1.src.rpm

Comment 12 Jochen Schmitt 2006-09-13 17:52:41 UTC
Good:
+ Local build works fine for FC-5.
+ Rpmlint has no complaints for binary rpm.
+ Rpmlint has no complaints for installed rpm.
+ Tar ball matched with upstream.
+ Binaries doesn't crash during startup.
+ Verbatin copy of license in %doc
+ License is GPLv2
+ Mock build works fine for FC-5

Your package is APPOVED !!1



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