[Bug 894338] Review Request: libdistorm - A lightweight, easy-to-use and fast disassembler/decomposer library for x86/AMD64

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=894338

Michael Schwendt <mschwendt@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mschwendt@xxxxxxxxx

--- Comment #6 from Michael Schwendt <mschwendt@xxxxxxxxx> ---
That can only be because you misunderstand Eric's review in comment 1 and my
comment 2.

More slowly then, okay. From comment 1, where fedora-review reported this
packaging failure:

  [!]: Development (unversioned) .so files in -devel subpackage, if present.
       Note: Unversioned so-files directly in %_libdir.

This is a false positive. You could not move /usr/lib64/libdistorm3.so to the
-devel package, because it is the only file in the base package. And it is a
run-time library, not a development file.

[...]

I see you've updated the spec file silently. Please bump the "Release" version
when you do that, and maintain the %changelog section, too.

The updated src.rpm still suffers from several issues. What you've changed
silently with regard to the shared library doesn't make sense.

Do run "rpmlint -i" on both the src.rpm and the built rpms. Also try a simple
"rpmbuild --rebuild" with your src.rpm. It cannot be built more than once
because of the weird things you do in %prep:

$ rpmbuild --rebuild libdistorm-3.3-1.fc18.src.rpm 
Installing libdistorm-3.3-1.fc18.src.rpm
warning: user rcvalle does not exist - using root
warning: group rcvalle does not exist - using root
warning: user rcvalle does not exist - using root
warning: group rcvalle does not exist - using root
Executing(%prep): /bin/sh -e /home/ms18b/tmp/rpm/tmp/rpm-tmp.lelXoK
+ umask 022
+ cd /home/ms18b/tmp/rpm/BUILD
+ unzip /home/ms18b/tmp/rpm/SOURCES/libdistorm-3.3/distorm3.zip
Archive:  /home/ms18b/tmp/rpm/SOURCES/libdistorm-3.3/distorm3.zip
replace distorm3/COPYING? [y]es, [n]o, [A]ll, [N]one, [r]ename: 
^C

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=I5GBwCTqOH&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]