View previous topic :: View next topic |
Author |
Message |
disi Veteran
Joined: 28 Nov 2003 Posts: 1354 Location: Out There ...
|
|
Back to top |
|
|
mrueg Retired Dev
Joined: 08 Jul 2012 Posts: 36 Location: Berlin, Deutschland
|
|
Back to top |
|
|
firefly Watchman
Joined: 31 Oct 2002 Posts: 5167
|
Posted: Sun Jul 08, 2012 10:17 am Post subject: |
|
|
im 2. patch passt was nicht.
wiso hast du das auf CPPFLAGS geändert obwohl hier nur c sourcen vorhanden sind?
_________________ Ein Ring, sie zu knechten, sie alle zu finden,
Ins Dunkel zu treiben und ewig zu binden
Im Lande Mordor, wo die Schatten drohn. |
|
Back to top |
|
|
disi Veteran
Joined: 28 Nov 2003 Posts: 1354 Location: Out There ...
|
|
Back to top |
|
|
mrueg Retired Dev
Joined: 08 Jul 2012 Posts: 36 Location: Berlin, Deutschland
|
Posted: Sun Jul 08, 2012 10:20 am Post subject: |
|
|
hab den patch mit hasufells hilfe noch etwas verbessert
CPPFLAGS sind ja Präprozessorflags, verwechselst du das mit CXXFLAGS? |
|
Back to top |
|
|
disi Veteran
Joined: 28 Nov 2003 Posts: 1354 Location: Out There ...
|
|
Back to top |
|
|
mrueg Retired Dev
Joined: 08 Jul 2012 Posts: 36 Location: Berlin, Deutschland
|
Posted: Sun Jul 08, 2012 10:29 am Post subject: |
|
|
Ich hatte es jetzt erstmal unter dev-util
aber ich denke sys-apps wäre auch passend |
|
Back to top |
|
|
firefly Watchman
Joined: 31 Oct 2002 Posts: 5167
|
Posted: Sun Jul 08, 2012 10:51 am Post subject: |
|
|
mrueg wrote: | hab den patch mit hasufells hilfe noch etwas verbessert
CPPFLAGS sind ja Präprozessorflags, verwechselst du das mit CXXFLAGS? |
meinst du mich? Wenn ja dann ist deine Aussage/frage falsch. CPPFLAGS/CXXFLAGS sind env variablen um Preprozessor variablen für den c++ compiler anzugeben, welche beim aufruf des compilers als parameter
übergeben werden. Da es sich bei radeontop um ein C programm handelt und nicht um ein c++ Programm muss es CFLAGS heißen und nicht CPPFLAGS/CXXFLAGS.
-> Der teil deines Patches ist falsch. _________________ Ein Ring, sie zu knechten, sie alle zu finden,
Ins Dunkel zu treiben und ewig zu binden
Im Lande Mordor, wo die Schatten drohn. |
|
Back to top |
|
|
mrueg Retired Dev
Joined: 08 Jul 2012 Posts: 36 Location: Berlin, Deutschland
|
Posted: Sun Jul 08, 2012 11:20 am Post subject: |
|
|
firefly wrote: | mrueg wrote: | hab den patch mit hasufells hilfe noch etwas verbessert
CPPFLAGS sind ja Präprozessorflags, verwechselst du das mit CXXFLAGS? |
meinst du mich? Wenn ja dann ist deine Aussage/frage falsch. CPPFLAGS/CXXFLAGS sind env variablen um Preprozessor variablen für den c++ compiler anzugeben, welche beim aufruf des compilers als parameter
übergeben werden. Da es sich bei radeontop um ein C programm handelt und nicht um ein c++ Programm muss es CFLAGS heißen und nicht CPPFLAGS/CXXFLAGS.
-> Der teil deines Patches ist falsch. |
Ja, das war auf deinen Post bezogen. Hätte ich vllt. mitquoten sollen.
Ich bin kein Experte, aber Wikipedia sagt dazu:
"CFLAGS allows to add switches for the C compiler, while CXXFLAGS is meant to be used when invoking a C++ compiler. Similarly, a variable CPPFLAGS exists with switches to be passed to the C preprocessor."
http://en.wikipedia.org/wiki/CFLAGS
Ist jetzt vielleicht nicht definitiv die beste Quelle, aber nach meiner Lesart ist CPPFLAGS auch für reinen C-Code zulässig? |
|
Back to top |
|
|
firefly Watchman
Joined: 31 Oct 2002 Posts: 5167
|
Posted: Sun Jul 08, 2012 11:33 am Post subject: |
|
|
mrueg wrote: |
Ja, das war auf deinen Post bezogen. Hätte ich vllt. mitquoten sollen.
Ich bin kein Experte, aber Wikipedia sagt dazu:
"CFLAGS allows to add switches for the C compiler, while CXXFLAGS is meant to be used when invoking a C++ compiler. Similarly, a variable CPPFLAGS exists with switches to be passed to the C preprocessor."
http://en.wikipedia.org/wiki/CFLAGS
Ist jetzt vielleicht nicht definitiv die beste Quelle, aber nach meiner Lesart ist CPPFLAGS auch für reinen C-Code zulässig? |
Ok stimmt CPPFLAGS kann für einen C und C++ compiler verwendet werden um preprocessor flags anzugeben. Da habe ich mich vertan. Nur wiso was ändern wenn das bestehende schon funktioniert? _________________ Ein Ring, sie zu knechten, sie alle zu finden,
Ins Dunkel zu treiben und ewig zu binden
Im Lande Mordor, wo die Schatten drohn. |
|
Back to top |
|
|
mrueg Retired Dev
Joined: 08 Jul 2012 Posts: 36 Location: Berlin, Deutschland
|
Posted: Sun Jul 08, 2012 12:06 pm Post subject: |
|
|
firefly wrote: | mrueg wrote: |
Ja, das war auf deinen Post bezogen. Hätte ich vllt. mitquoten sollen.
Ich bin kein Experte, aber Wikipedia sagt dazu:
"CFLAGS allows to add switches for the C compiler, while CXXFLAGS is meant to be used when invoking a C++ compiler. Similarly, a variable CPPFLAGS exists with switches to be passed to the C preprocessor."
http://en.wikipedia.org/wiki/CFLAGS
Ist jetzt vielleicht nicht definitiv die beste Quelle, aber nach meiner Lesart ist CPPFLAGS auch für reinen C-Code zulässig? |
Ok stimmt CPPFLAGS kann für einen C und C++ compiler verwendet werden um preprocessor flags anzugeben. Da habe ich mich vertan. Nur wiso was ändern wenn das bestehende schon funktioniert? |
Richtig es funktioniert auch ohne, aber ENABLE_NLS ist ein Präprozessormakro und sollte somit in CPPFLAGS gehören.
"A C preprocessor macro can be used to detect these two cases. Usually, when libintl.h was found and not explicitly disabled, the ENABLE_NLS macro will be defined to 1 in the autoconf generated configuration file (usually called config.h). In the two negative situations, however, this macro will not be defined, thus it will evaluate to 0 in C preprocessor expressions. "
http://www.gnu.org/software/gettext/manual/gettext.html
hier die aktuelle Version des Ebuilds+Patches, die ich mit hasufells Hilfe erstellt habe:
http://bpaste.net/show/33989/ http://bpaste.net/show/33990/ |
|
Back to top |
|
|
firefly Watchman
Joined: 31 Oct 2002 Posts: 5167
|
Posted: Sun Jul 08, 2012 12:15 pm Post subject: |
|
|
mrueg wrote: |
Richtig es funktioniert auch ohne, aber ENABLE_NLS ist ein Präprozessormakro und sollte somit in CPPFLAGS gehören. |
Sollte ist aber kein muss. Und durch deine Änderung wird diese Präprozessormakro nie gesetzt. Da diese env variable beim eigentlichen Aufruf des compilers nicht mit angegeben wird. Denn der compiler selbst wertet diese env-variablen nicht aus, sondern werden vom entsprechenden build-system ausgewertet. Im falle eines simplen Makefiles muss dies der Ersteller des Makefiles selbst machen.
Edit: in deinem letzten patch wird diese Var mit angegebenbeim Aufruf des compilers, mein Fehler. _________________ Ein Ring, sie zu knechten, sie alle zu finden,
Ins Dunkel zu treiben und ewig zu binden
Im Lande Mordor, wo die Schatten drohn. |
|
Back to top |
|
|
V10lator Apprentice
Joined: 11 Jul 2004 Posts: 207
|
|
Back to top |
|
|
|