Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 887633 - [guru] dev-util/fpm-0.7.0-r1 installs less files with clang stricter
Summary: [guru] dev-util/fpm-0.7.0-r1 installs less files with clang stricter
Status: RESOLVED FIXED
Alias: None
Product: GURU
Classification: Unclassified
Component: Package issues (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Sergey Torokhov
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-12-21 09:58 UTC by Agostino Sarubbo
Modified: 2022-12-27 08:05 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2022-12-21 09:58:46 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: dev-util/fpm-0.7.0-r1 installs less files with clang stricter.
Discovered on: amd64 (internal ref: guru_tinderbox)

NOTE:
To reproduce this issue:
1) Install the package with CC=clang - CXX=clang++
2) Put the list of the installed files into a file ( qlist -e PACKAGE:SLOT | sort | uniq > /tmp/qlist1 )
3) Install the package with CC=clang - CXX=clang++ - CFLAGS/CXXFLAGS=-Werror=implicit-function-declaration -Werror=implicit-int -Werror=incompatible-function-pointer-types
4) Put the list of the installed files into a file ( qlist -e PACKAGE:SLOT | sort | uniq > /tmp/qlist2 )
5) See the diff between the two files ( diff -ru /tmp/qlist_1 /tmp/qlist_2 )

On this machine:
- the installed files with clang are 4
- the installed files with clang stricter are 3

This type of report comes from a check that is a bit outside of the tinderbox, that's the reason because build.log was not provided (however in cases like these the build.log doesn't help much) but if you need further help to diagnose this issue feel free to ask.

In any case I'm providing a diff.
Comment 1 Agostino Sarubbo gentoo-dev 2022-12-21 09:59:08 UTC
DIFF:
--- /dev/fd/63  2022-12-21 05:48:58.407720644 +0100
+++ /dev/fd/62  2022-12-21 05:48:58.407720644 +0100
@@ -1,4 +1,3 @@
-/usr/bin/fpm
 /usr/share/doc/fpm-0.7.0-r1/LICENSE.bz2
 /usr/share/doc/fpm-0.7.0-r1/PACKAGING.md.bz2
 /usr/share/doc/fpm-0.7.0-r1/README.md.bz2
Comment 2 Sergey Torokhov 2022-12-21 13:17:48 UTC
I'm not able to reproduce it with clang-15.0.6 both with x86_64-pc-linux-gnu-ar and llvm-ar.

Could you please provide additional info what clang version is used?

To get more detailed build log the commands within .ebuild 'fpm build' and 'fpm install' could be replaced with 'fpm build --verbose' and 'fpm install --verbose' respectively.

!!!
But if I set FCFLAGS as mentioned above CFLAGS/CXXFLAGS values then I got warnings that '-Werror=implicit-function-declaration' and '-Werror=implicit-int' aren't valid for Fortran and finnaly got error:"-Werror=incompatible-function-pointer-types: no option '-Wincompatible-function-pointer-types'; did you mean '-Wincompatible-pointer-types'".

It seems there is an issue of incompatibility of new clang CFLAGS/CXXFLAGS (the same error if build with gcc instead of clang with mentioned flags).
Comment 3 Sergey Torokhov 2022-12-21 13:38:55 UTC
At first I must to add "|| die" at the end of lines with external commands and maybe add "--verbose" option however without it the error about compiler "no option" is reported.
Comment 4 Larry the Git Cow gentoo-dev 2022-12-27 08:05:47 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=3b8d080ae6a76fcf463a77a145e0b03b6eceb7a4

commit 3b8d080ae6a76fcf463a77a145e0b03b6eceb7a4
Author:     Sergey Torokhov <torokhov-s-a@yandex.ru>
AuthorDate: 2022-12-22 01:09:05 +0000
Commit:     Sergey Torokhov <torokhov-s-a@yandex.ru>
CommitDate: 2022-12-22 01:09:05 +0000

    dev-util/fpm: call 'die' if fail; add '--verbose' for detailed build.log
    
    Closes: https://bugs.gentoo.org/887633
    
    Signed-off-by: Sergey Torokhov <torokhov-s-a@yandex.ru>

 dev-util/fpm/fpm-0.7.0-r1.ebuild | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)