[GitHub] [commons-daemon] RayWang10 opened a new pull request #16: psupport.m4: remove incorrect definition

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [commons-daemon] RayWang10 opened a new pull request #16: psupport.m4: remove incorrect definition

GitBox
RayWang10 opened a new pull request #16: psupport.m4: remove incorrect definition
URL: https://github.com/apache/commons-daemon/pull/16
 
 
   Fix jsvc build error on s390, arm, aarch64, mipsel, mips. jni_md.h file
   could not be found, due to incorrent definition ‘supported_os’. it should
   not be defined as the architecture of the cpu.
   
   Because I don't know how to modify the previous commit,so i gave up the previous pull request, and resubmitted.- -|

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services
Reply | Threaded
Open this post in threaded view
|

[GitHub] [commons-daemon] RayWang10 commented on issue #16: psupport.m4: remove incorrect definition

GitBox
RayWang10 commented on issue #16: psupport.m4: remove incorrect definition
URL: https://github.com/apache/commons-daemon/pull/16#issuecomment-586128593
 
 
   build log:
   <pre>
   (cd native; make  all)
   make[1]: Entering directory '/home/ray/tmp/ttt/src/native/unix/native'
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c jsvc-unix.c -o jsvc-unix.o
   jsvc-unix.c: In function 'run_controller':
   jsvc-unix.c:1293:20: warning: assignment to '__sighandler_t' {aka 'void (*)(int)'} from incompatible pointer type 'void (*)(int,  siginfo_t *, void *)' {aka 'void (*)(int,  struct <anonymous> *, void *)'} [-Wincompatible-pointer-types]
    1293 |     act.sa_handler = controller;
         |                    ^
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c arguments.c -o arguments.o
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c debug.c -o debug.o
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c dso-dlfcn.c -o dso-dlfcn.o
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c dso-dyld.c -o dso-dyld.o
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c help.c -o help.o
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c home.c -o home.o
   gcc -g -O2 -DOS_LINUX -DDSO_DLFCN -DCPU=\"mipsel\" -Wall -Wstrict-prototypes   -I/usr/lib/jvm/default-java/include -I/usr/lib/jvm/default-java/include/mipsel -c java.c -o java.o
   In file included from java.c:23:
   /usr/lib/jvm/default-java/include/jni.h:45:10: fatal error: jni_md.h: No such file or directory
      45 | #include "jni_md.h"
         |          ^~~~~~~~~~
   compilation terminated.
   make[1]: *** [../Makedefs:31: java.o] Error 1
   make[1]: Leaving directory '/home/ray/tmp/ttt/src/native/unix/native'
   make: *** [Makefile:23: native/all] Error 2
   </pre>
   
   the jni_md.h file path: /usr/lib/jvm/default-java/include/linux/

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services
Reply | Threaded
Open this post in threaded view
|

[GitHub] [commons-daemon] RayWang10 commented on issue #16: psupport.m4: remove incorrect definition

GitBox
In reply to this post by GitBox
RayWang10 commented on issue #16: psupport.m4: remove incorrect definition
URL: https://github.com/apache/commons-daemon/pull/16#issuecomment-586788380
 
 
   > I've no way of confirming this patch as I don't have access to the hardware but logically, it makes sense to me. Can you add an entry for this to the change-log please.
   
   sorry, i don't know what this mean "add an entry for this to the change-log". how can i do it?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services
Reply | Threaded
Open this post in threaded view
|

[GitHub] [commons-daemon] markt-asf commented on issue #16: psupport.m4: remove incorrect definition

GitBox
In reply to this post by GitBox
markt-asf commented on issue #16: psupport.m4: remove incorrect definition
URL: https://github.com/apache/commons-daemon/pull/16#issuecomment-586889539
 
 
   The change log is recorded in this file:
   https://github.com/apache/commons-daemon/blob/master/src/changes/changes.xml
   
   You need to add an entry to that file for this change. It should be added to the end to the section for 1.2.3. For `dev` use `markt` and for `due-to` use your name, github id or similar.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services
Reply | Threaded
Open this post in threaded view
|

[GitHub] [commons-daemon] markt-asf merged pull request #16: psupport.m4: remove incorrect definition

GitBox
In reply to this post by GitBox
markt-asf merged pull request #16: psupport.m4: remove incorrect definition
URL: https://github.com/apache/commons-daemon/pull/16
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services