Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 725766 - sys-apps/salinfo fails to compile
Summary: sys-apps/salinfo fails to compile
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: IA-64 team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-28 07:19 UTC by Agostino Sarubbo
Modified: 2020-05-31 10:25 UTC (History)
0 users

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


Attachments
build.log (build.log,7.93 KB, text/plain)
2020-05-28 07:19 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 2020-05-28 07:19:48 UTC
This is an auto-filed bug because sys-apps/salinfo fails to compile.
The issue was originally discovered on amd64, but it may be reproducible on other arches as well.
If you think that a different summary clarifies the issue better, feel free to change it.
Attached build log and emerge --info.

NOTE:
If you need further logs, feel free to ask.
Comment 1 Agostino Sarubbo gentoo-dev 2020-05-28 07:19:53 UTC
Created attachment 642210 [details]
build.log

build log and emerge --info
Comment 2 Sergei Trofimovich (RETIRED) gentoo-dev 2020-05-31 10:17:00 UTC
(In reply to Agostino Sarubbo from comment #0)
> The issue was originally discovered on amd64, but it may be reproducible on
> other arches as well.

I think SAL is ia64-specific interface to the system. How did you get it to be keyworded on amd64? It has no keywords outside ia64:

$ eshowkw salinfo
Keywords for sys-apps/salinfo:
    |                             |   u   |
    | a   a     p s     a   r     |   n   |
    | m   r h   p p   s l i i m m | e u s | r
    | d a m p p c a x 3 p a s 6 i | a s l | e
    | 6 r 6 p p 6 r 8 9 h 6 c 8 p | p e o | p
    | 4 m 4 a c 4 c 6 0 a 4 v k s | i d t | o
----+-----------------------------+-------+-------
1.2 | o o o o o o o o o o ~ o o o | 6 o 0 | gentoo

"""
x86_64-pc-linux-gnu-gcc -O2 -pipe -march=native -frecord-gcc-switches -Wall   -Wl,-O1 -Wl,--as-needed -Wl,--defsym=__gentoo_check_ldflags__=0  salinfo_decode_oem.c   -o salinfo_decode_oem
salinfo_decode_oem.c:24:10: fatal error: asm/fpu.h: No such file or directory
   24 | #include <asm/fpu.h>
      |          ^~~~~~~~~~~
compilation terminated.
make: *** [<builtin>: salinfo_decode_oem] Error 1
make: *** Waiting for unfinished jobs....
In file included from mca.h:29,
                 from salinfo_decode.c:57:
sal.h:30:10: fatal error: asm/fpu.h: No such file or directory
   30 | #include <asm/fpu.h>
      |          ^~~~~~~~~~~
compilation terminated.
make: *** [<builtin>: salinfo_decode.o] Error 1
In file included from mca.h:29,
                 from mca.c:59:
sal.h:30:10: fatal error: asm/fpu.h: No such file or directory
   30 | #include <asm/fpu.h>
      |          ^~~~~~~~~~~
compilation terminated.
make: *** [<builtin>: mca.o] Error 1
"""
Comment 3 Agostino Sarubbo gentoo-dev 2020-05-31 10:21:43 UTC
sorry for the spam
Comment 4 Sergei Trofimovich (RETIRED) gentoo-dev 2020-05-31 10:22:21 UTC
Also no chance to compile outside ia64:

fpu.h is provided only in alpha and ia64:

"""
$ find -name fpu.h | fgrep uapi
./arch/alpha/include/uapi/asm/fpu.h
./arch/ia64/include/uapi/asm/fpu.h
"""

and ia64_fpreg is only ia64-specific thing:

"""
  127 |  struct ia64_fpreg fr[128];
      |                    ^~
"""
Comment 5 Sergei Trofimovich (RETIRED) gentoo-dev 2020-05-31 10:24:35 UTC
(In reply to Agostino Sarubbo from comment #3)
> sorry for the spam

Oh, thats ok. We can express platform dependence explicitly as well.
Comment 6 Larry the Git Cow gentoo-dev 2020-05-31 10:25:59 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ae73af2a69c9f381aa0139e051041d02f8c494b5

commit ae73af2a69c9f381aa0139e051041d02f8c494b5
Author:     Sergei Trofimovich <slyfox@gentoo.org>
AuthorDate: 2020-05-31 10:25:43 +0000
Commit:     Sergei Trofimovich <slyfox@gentoo.org>
CommitDate: 2020-05-31 10:25:54 +0000

    sys-apps/salinfo: mark KEYWORDS as ia64-only, bug #725766
    
    Also add a comment around KEYWORDS why it has little chance
    to work anywhere else.
    
    Reported-by: Agostino Sarubbo
    Closes: https://bugs.gentoo.org/725766
    Package-Manager: Portage-2.3.100, Repoman-2.3.22
    Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>

 sys-apps/salinfo/salinfo-1.2.ebuild | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)