[Bug 2023061] Review Request: libdeflate - Fast implementation of DEFLATE, zlib, and gzip

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=2023061



--- Comment #10 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
Created attachment 1842971
  --> https://bugzilla.redhat.com/attachment.cgi?id=1842971&action=edit
patch to add missing debuginfo and fix stripping

> Got the "shlib-policy-name-error" resolved by renaming to libdeflate0. 

That's a strange one. I've never seen this message before. Are you using a
non-Fedora version of rpmlint?
Fedora does not name packages after so-versions, please rename it back.

> %define debug_package %{nil}

This is wrong, we need debuginfo.

> CFLAGS="-fpic -pie -g"

This is also wrong. Packages must use standard compilation flags as provided by
the macros.
[https://docs.fedoraproject.org/en-US/packaging-guidelines/#_compiler_flags]

> %{__strip}

General rule: do not use double-underscore-prefixed commands. Those were
popular back in the day,
but they are useless and the guidelines finally have been changed to discourage
them.

Specific rule: strip is called on all binaries through
/usr/lib/rpm/redhat/brp-strip-lto,
so we need to figure out why that call doesn't work, and fix the compilation
process so that
it does.

> I dropped the static library package after noticing that most libraries on Fedora don't appear to ship their .a

Yeah, that's probably a good solution. I assumed you want the the static
library for something,
so I didn't suggest dropping it. But if it isn't needed for something else, it
can go.

Removing it does not solve the original issue: the debug data is missing. Let's
see what is needed to fix this.
...
Oh, it turns out that that this is the typical case of "upstream invents own
build system, the
result is 90% correct". Please see the attached patch.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2023061
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux