[Bug 2076374] Review Request: trafficserver - Fast, scalable and extensible HTTP/1.1 and HTTP/2 caching proxy server

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

 



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



--- Comment #5 from Neal Gompa <ngompa13@xxxxxxxxx> ---
Initial spec review:

> # Upstream does not support 32-bit architectures:
> # https://github.com/apache/trafficserver/issues/4432
> ExcludeArch:    armhfp

Please use "%{arm} %{ix86}" here (which expands to all 32-bit arm and x86
arches), "armhfp" isn't a real architecture.

(I'd prefer to use "%{arm32}" for 32-bit arm arches, but that only exists in
RHEL9+ right now, and nobody backported it to EPEL 8 yet...)

> # trafficserver does not work properly with OpenSSL 3.0.2 yet
> # Upstream issue: https://github.com/apache/trafficserver/issues/7341
> %if 0%{?fedora} >= 36 || 0%{?rhel} >= 9
> BuildRequires:  openssl1.1-devel
> %else
> BuildRequires:  openssl-devel
> %endif
[...]
> %if 0%{?fedora} >= 36 || 0%{?rhel} >= 9
> Requires:  openssl1.1
> %else
> Requires:  openssl
> # Require an OpenSSL which supports PROFILE=SYSTEM
> Conflicts:      openssl-libs < 1:1.0.1h-4
> %endif

openssl1.1-devel does not exist in RHEL/EPEL 9, so you cannot build for RHEL 9.
The situation around openssl1.1 is weird. There's compat-openssl11 in RHEL 9,
but it doesn't provide devel content, and it's unusually crippled (no FIPS
support, for example). I'm not sure what we should do in EPEL about it.

>   --enable-layout=Gentoo \

Umm, what?

> %ghost %verify(not md5 size mode mtime) %{_sharedstatedir}/selinux/%{selinuxtype}/active/modules/200/%{modulename}

The %verify stanza is redundant for a %ghost file (which already doesn't do any
verification)


-- 
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=2076374
_______________________________________________
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