https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Bug ID: 2315607 Summary: Review Request: python-geopmpy - Python bindings for libgeopm Product: Fedora Version: rawhide OS: Linux Status: NEW Component: Package Review Severity: medium Assignee: nobody@fedoraproject.org Reporter: aekoroglu@linux.intel.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
SPEC Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/geopm/fedora-ra... SRPM Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/geopm/fedora-ra...
Description: The Global Extensible Open Power Manager (GEOPM) provides a framework to explore power and energy optimizations on platforms with heterogeneous mixes of computing hardware.
Users can monitor their system's energy and power consumption, and safely optimize system hardware settings to achieve energy efficiency and/or performance objectives.
Reproducible: Always
https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://geopm.github.io
--- Comment #1 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/8091492 (failed)
Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please make sure the package builds successfully at least for Fedora Rawhide.
- If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field
--- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service
If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Ali Erdinc Koroglu aekoroglu@linux.intel.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |2315604, 2315605, 2315606 Doc Type|--- |If docs needed, set a value
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2315604 [Bug 2315604] Review Request: libgeopmd - C/C++ implementation of the GEOPM access service https://bugzilla.redhat.com/show_bug.cgi?id=2315605 [Bug 2315605] Review Request: libgeopm - C/C++ implementation of the GEOPM runtime service https://bugzilla.redhat.com/show_bug.cgi?id=2315606 [Bug 2315606] Review Request: geopmd - GEOPM daemon
https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |negativo17@gmail.com Assignee|nobody@fedoraproject.org |negativo17@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? Status|NEW |ASSIGNED
--- Comment #2 from Simone Caronni negativo17@gmail.com --- Mixed use of spaces and tabs, starting line 12-13.
'Name:' is misaligned with the rest of the spec file, optional.
Can you please build the package with the automatic build and runtime requirements? https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_provides_...
It's very easy to do, for example: https://github.com/negativo17/python-sabctools/blob/master/python-sabctools....
https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(ali.erdinc.korogl | |u@intel.com) CC| |ali.erdinc.koroglu@intel.co | |m
--- Comment #3 from Simone Caronni negativo17@gmail.com --- @ali.erdinc.koroglu@intel.com can we proceed with the review? It has been a month.
Thanks.
https://bugzilla.redhat.com/show_bug.cgi?id=2315607 Bug 2315607 depends on bug 2315604, which changed state.
Bug 2315604 Summary: Review Request: libgeopmd - C/C++ implementation of the GEOPM access service https://bugzilla.redhat.com/show_bug.cgi?id=2315604
What |Removed |Added ---------------------------------------------------------------------------- Status|RELEASE_PENDING |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=2315607 Bug 2315607 depends on bug 2315606, which changed state.
Bug 2315606 Summary: Review Request: geopmd - GEOPM daemon https://bugzilla.redhat.com/show_bug.cgi?id=2315606
What |Removed |Added ---------------------------------------------------------------------------- Status|RELEASE_PENDING |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=2315607 Bug 2315607 depends on bug 2315605, which changed state.
Bug 2315605 Summary: Review Request: libgeopm - C/C++ implementation of the GEOPM runtime service https://bugzilla.redhat.com/show_bug.cgi?id=2315605
What |Removed |Added ---------------------------------------------------------------------------- Status|RELEASE_PENDING |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=2315607
Ali Erdinc Koroglu aekoroglu@linux.intel.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(ali.erdinc.korogl |needinfo- |u@intel.com) |needinfo?(negativo17@gmail. | |com)
--- Comment #4 from Ali Erdinc Koroglu aekoroglu@linux.intel.com --- Hello Simone, there are some incompatibilities when using python-macros with the upstream code. If it’s okay with you, we can complete the packaging using the old method for now. I’ll work on fixing the upstream issues and updating the spec file as soon as possible.
package-review@lists.fedoraproject.org
-
bugzilla＠redhat.com