[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

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

 



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



--- Comment #14 from Jaroslav Škarvada <jskarvad@xxxxxxxxxx> ---
(In reply to Ali Erdinc Koroglu from comment #12)
- Regarding the patch, maybe upstream could allow installation without
drivers/helpers, but for now I am OK with the patch.

- %make_build CFLAGS="%optflags -pie -Wl,-z,now" -C helpers/ndl-helper
Are the extra CFLAGS needed?

For the linker flags (-Wl,-z,now) I think it should use full distro's LDGFLAGS,
i.e. %{build_ldflags}. Maybe the upstream makefiles could be improved to obey
LDFLAGS variable. Then it may work out of the box without extra flags.

- it seems there is a circular dep i.e. python3-wult requires wult and wult
requires python3-wult. It means both packages have to be installed
simultaneously so is there any reason for having two packages?


-- 
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=2099771
_______________________________________________
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, report it: https://pagure.io/fedora-infrastructure/new_issue




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

  Powered by Linux