Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 933186 - [guru] app-admin/hardinfo2-2.2.4 overwrites CFLAGS/CXXFLAGS or adds uncommon ones
Summary: [guru] app-admin/hardinfo2-2.2.4 overwrites CFLAGS/CXXFLAGS or adds uncommon ...
Status: RESOLVED FIXED
Alias: None
Product: GURU
Classification: Unclassified
Component: Package issues (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Quincy Fleming
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: qa-guru
  Show dependency tree
 
Reported: 2024-05-30 10:22 UTC by Agostino Sarubbo
Modified: 2024-11-27 16:35 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,275.10 KB, text/plain)
2024-05-30 10:22 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2024-05-30 10:22:45 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: app-admin/hardinfo2-2.1.11 overwrites CFLAGS/CXXFLAGS or adds uncommon ones.
Discovered on: amd64 (internal ref: guru_ci)

Info about the issue:
https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#QA0069
Comment 1 Agostino Sarubbo gentoo-dev 2024-05-30 10:22:46 UTC
Created attachment 894650 [details]
build.log

build log and emerge --info
Comment 2 Quincy Fleming 2024-05-30 19:41:28 UTC
Can we make an exception for this?  The developer insists optimization flags should be turned off just for compiling these parts of the code as defined here in CMakeLists.txt:313-320

set_source_files_properties(
	modules/benchmark/blowfish.c
	modules/benchmark/fftbench.c
	modules/benchmark/md5.c
	modules/benchmark/sha1.c
	PROPERTIES
	COMPILE_FLAGS "-O0 -Wall -Wextra -Werror=implicit-function-declaration"
)

The developer states the reason in modules/benchmark/md5.c:34-36 and in
modules/benchmark/sha1.c:36-38

#if defined(__OPTIMIZE__)
#error You must compile this program without "-O". (Or else the benchmark results may be different!)
#endif

which will cause the program to not compile if optimizations are turned on for this part of the code. I can just remove those 3 lines of code from both modules/benchmark/md5.c and modules/benchmark/sha1.c, and remove the -O0 flag from CMakeLists.txt and it will compile just fine but benchmark results may be different as the developer states.
Comment 3 Agostino Sarubbo gentoo-dev 2024-08-06 07:11:43 UTC
guru_ci has reproduced this issue with version 2.1.11-r1 - Updating summary.
Comment 4 Agostino Sarubbo gentoo-dev 2024-09-09 12:54:01 UTC
guru_ci has reproduced this issue with version 2.1.14 - Updating summary.
Comment 5 Agostino Sarubbo gentoo-dev 2024-10-04 07:08:36 UTC
guru_ci has reproduced this issue with version 2.1.17 - Updating summary.
Comment 6 Agostino Sarubbo gentoo-dev 2024-11-03 08:06:09 UTC
guru_ci has reproduced this issue with version 2.2.1 - Updating summary.
Comment 7 Agostino Sarubbo gentoo-dev 2024-11-09 07:09:06 UTC
guru_ci has reproduced this issue with version 2.2.4 - Updating summary.
Comment 8 Larry the Git Cow gentoo-dev 2024-11-27 16:35:24 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=019e8b6181b183fd812117b3029591ae88661528

commit 019e8b6181b183fd812117b3029591ae88661528
Author:     Quincy Fleming <quincyf467@protonmail.com>
AuthorDate: 2024-11-27 11:30:10 +0000
Commit:     Quincy Fleming <quincyf467@protonmail.com>
CommitDate: 2024-11-27 11:30:10 +0000

    app-admin/hardinfo2: drop 2.2.4-r0
    
    Closes: https://bugs.gentoo.org/933186
    Signed-off-by: Quincy Fleming <quincyf467@protonmail.com>

 app-admin/hardinfo2/hardinfo2-2.2.4.ebuild | 60 ------------------------------
 1 file changed, 60 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=484deb3ab9e3e1014c54f872735611a9a2d09009

commit 484deb3ab9e3e1014c54f872735611a9a2d09009
Author:     Quincy Fleming <quincyf467@protonmail.com>
AuthorDate: 2024-11-27 11:29:18 +0000
Commit:     Quincy Fleming <quincyf467@protonmail.com>
CommitDate: 2024-11-27 11:29:18 +0000

    app-admin/hardinfo2: Fix overwriting CFLAGS
    
    Bug: https://bugs.gentoo.org/933186
    Signed-off-by: Quincy Fleming <quincyf467@protonmail.com>

 app-admin/hardinfo2/hardinfo2-2.2.4-r1.ebuild | 62 +++++++++++++++++++++++++++
 1 file changed, 62 insertions(+)