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

Summary: Review Request: luks-tools
Product: [Fedora] Fedora Reporter: W. Michael Petullo <redhat>
Component: Package ReviewAssignee: Jochen Schmitt <jochen>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: astrand, opensource
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-09-13 22:04:02 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:
Bug Depends On:    
Bug Blocks: 163779, 166035    
Attachments:
Description Flags
Make luks-setup build clean on FC5 none

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