Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 777375 - mate-extra/mate-sensors-applet-1.24.1 - i2c-proc-plugin.c:147:41: error: 'LC_NUMERIC' undeclared (first use in this function) (on musl)
Summary: mate-extra/mate-sensors-applet-1.24.1 - i2c-proc-plugin.c:147:41: error: 'LC_...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo MATE Desktop
URL: https://github.com/mate-desktop/mate-...
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks: musl-porting
  Show dependency tree
 
Reported: 2021-03-20 12:24 UTC by ernsteiswuerfel
Modified: 2022-07-15 01:56 UTC (History)
1 user (show)

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


Attachments
build.log (mate-sensors-applet-1.24.1:20210320-023430.log,69.44 KB, text/plain)
2021-03-20 12:24 UTC, ernsteiswuerfel
Details
emerge --info (file_777375.txt,5.87 KB, text/plain)
2021-03-20 12:24 UTC, ernsteiswuerfel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ernsteiswuerfel archtester 2021-03-20 12:24:27 UTC
Created attachment 692562 [details]
build.log

[...]
make[2]: Entering directory '/var/tmp/portage/mate-extra/mate-sensors-applet-1.24.1/work/mate-sensors-applet-1.24.1/plugins/i2c-proc'
/bin/sh ../../libtool  --tag=CC   --mode=compile powerpc64-gentoo-linux-musl-gcc -DHAVE_CONFIG_H -I. -I../../sensors-applet  -DMATELOCALEDIR=\""/usr/share/locale/"\" -DG_LOG_DOMAIN=\""sensors-applet"\" -DPIXMAPS_DIR=\""/usr/share/pixmaps/mate-sensors-applet/"\" -DDATADIR=\""/usr/share"\" -DLIBDIR=\""/usr/lib"\" -DSYSCONFDIR=\""/etc"\" -DPREFIX=\""/usr"\" -I../.. -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi/include -pthread -I/usr/include/libmount -I/usr/include/blkid    -Os -mcpu=970 -pipe -c -o i2c-proc-plugin.lo i2c-proc-plugin.c
libtool: compile:  powerpc64-gentoo-linux-musl-gcc -DHAVE_CONFIG_H -I. -I../../sensors-applet -DMATELOCALEDIR=\"/usr/share/locale/\" -DG_LOG_DOMAIN=\"sensors-applet\" -DPIXMAPS_DIR=\"/usr/share/pixmaps/mate-sensors-applet/\" -DDATADIR=\"/usr/share\" -DLIBDIR=\"/usr/lib\" -DSYSCONFDIR=\"/etc\" -DPREFIX=\"/usr\" -I../.. -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi/include -pthread -I/usr/include/libmount -I/usr/include/blkid -Os -mcpu=970 -pipe -c i2c-proc-plugin.c  -fPIC -DPIC -o .libs/i2c-proc-plugin.o
i2c-proc-plugin.c: In function 'i2c_proc_plugin_get_sensor_value':
i2c-proc-plugin.c:147:31: warning: implicit declaration of function 'setlocale' [-Wimplicit-function-declaration]
  147 |     if (NULL == (old_locale = setlocale(LC_NUMERIC, "C"))) {
      |                               ^~~~~~~~~
i2c-proc-plugin.c:147:41: error: 'LC_NUMERIC' undeclared (first use in this function)
  147 |     if (NULL == (old_locale = setlocale(LC_NUMERIC, "C"))) {
      |                                         ^~~~~~~~~~
i2c-proc-plugin.c:147:41: note: each undeclared identifier is reported only once for each function it appears in
Comment 1 ernsteiswuerfel archtester 2021-03-20 12:24:57 UTC
Created attachment 692565 [details]
emerge --info
Comment 2 Larry the Git Cow gentoo-dev 2022-07-15 01:56:40 UTC
The bug has been closed via the following commit(s):

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

commit 1bd3fa116f8bbf209550c8bfef2bc4ffd72ccc2e
Author:     brahmajit das <brahmajit.xyz@gmail.com>
AuthorDate: 2022-07-02 18:50:28 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-07-15 01:51:09 +0000

    mate-extra/mate-sensors-applet: Removing unnecessary config checks
    
    Removing unnecessary HAVE_STDIO_H and HAVE_LOCALE_H config checks. The
    HAVE_LOCALE_H also makes fails to build on musl system.
    
    Upstream issue: mate-desktop/mate-sensors-applet#123
    Closes: https://bugs.gentoo.org/777375
    Signed-off-by: brahmajit das <brahmajit.xyz@gmail.com>
    Closes: https://github.com/gentoo/gentoo/pull/26197
    Signed-off-by: Sam James <sam@gentoo.org>

 ...ensors-applet-1.24.1-dont-check-for-headers.patch | 20 ++++++++++++++++++++
 .../mate-sensors-applet-1.24.1.ebuild                |  4 ++++
 2 files changed, 24 insertions(+)