[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 #6 from Jered Floyd <jered@xxxxxxxxxx> ---
(In reply to Neal Gompa from comment #5)

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

Will do.

> > # 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
> 
> 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.

Ideally we fix this upstream so we can use OpenSSL 3.0 -- I haven't looked into
this yet.  Are you recommending I remove the "|| 0%{?rhel} >= 9" condition for
now?


> >   --enable-layout=Gentoo \
> 
> Umm, what?

It's not possible to get all the files installed in the right locations with
command-line flags to configure, and the "Gentoo" layout for config.layout is
exactly correct.  If you'd prefer I can duplicate the config stanza and call it
redhat?  I fretted over that for a few minutes.  See upstream:
https://github.com/apache/trafficserver/blob/master/config.layout


> > %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)

This is directly from the Fedora doc here:
 https://fedoraproject.org/wiki/SELinux/IndependentPolicy#The_.25files_Section

Are you suggesting a change to just:
  %ghost
%{_sharedstatedir}/selinux/%{selinuxtype}/active/modules/200/%{modulename}

?


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
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