> On Apr 12, 2018, at 11:54 21AM, Iris Clark <[email protected]> wrote: > > Hi. > > I believe that the internal page Christian references is for the test system. > > If you want to know whether the push arrived in the repository, you could > subscribe to [email protected]. The archive of recent push > notifications is public: > > > http://mail.openjdk.java.net/pipermail/jdk-submit-changes/2018-April/thread.html > > I wonder if the test system could be enhanced to send a brief notification > when a job is queued?
I’ve opened an enhancement for adding a notification once the job has been handed off to our build and test farm. Thanks, Christian > > Thanks, > Iris > > -----Original Message----- > From: Christian Tornqvist > Sent: Thursday, April 12, 2018 6:12 AM > To: Baesken, Matthias <[email protected]> > Cc: [email protected]; Alexey Ivanov <[email protected]>; > Doerr, Martin <[email protected]>; build-dev <[email protected]> > Subject: Re: 8201226 missing JNIEXPORT / JNICALL at some places in function > declarations/implementations - was : RE: missing JNIEXPORT / JNICALL at some > places in function declarations/implementations > > > >> On Apr 12, 2018, at 9:07 48AM, Baesken, Matthias <[email protected]> >> wrote: >> >>> Your submit job ran without failures, we were doing maintenance on >>> the jdk- submit repo yesterday and had turned off notifications. >>> Sorry for the inconvenience. >> >> Hi Christian , Thanks for the information about the submit job success. >> >> Is there a way to check (e.g. webpage) that a submit job has "arrived" >> and is queued for build/test ? > > Unfortunately that webpage is only available internally at this point, we > could look into sending an email notification that the job has been started > if that would help? > > Thanks, > Christian > >> Would have been helpful in this situation . >> >> Best regards, Matthias >> >> >>> -----Original Message----- >>> From: Christian Tornqvist [mailto:[email protected]] >>> Sent: Donnerstag, 12. April 2018 14:58 >>> To: Baesken, Matthias <[email protected]> >>> Cc: Alexey Ivanov <[email protected]>; Magnus Ihse Bursie >>> <[email protected]>; build-dev <build- >>> [email protected]>; Doerr, Martin <[email protected]> >>> Subject: Re: 8201226 missing JNIEXPORT / JNICALL at some places in >>> function declarations/implementations - was : RE: missing JNIEXPORT / >>> JNICALL at some places in function declarations/implementations >>> >>> Hi Matthias, >>> >>> >>>> On Apr 12, 2018, at 3:49 35AM, Baesken, Matthias >>> <[email protected]> wrote: >>>> >>>> Hi, could someone please sponsor the change now ? >>>> >>>> And could someone please check what happened to the submit-repo ? >>>> Yesterday I pushed to the submit repo to check my change , but no >>> response so far . >>>> Maybe the submit repo is not working currently , not sure about it . >>> >>> Your submit job ran without failures, we were doing maintenance on >>> the jdk- submit repo yesterday and had turned off notifications. >>> Sorry for the inconvenience. >>> >>> Thanks, >>> Christian >>>> >>>> >>>> Best regards , Matthias >>>> >>>> >>>> >>>> >>>>> -----Original Message----- >>>>> From: Baesken, Matthias >>>>> Sent: Mittwoch, 11. April 2018 11:20 >>>>> To: 'Alexey Ivanov' <[email protected]>; Magnus Ihse Bursie >>>>> <[email protected]> >>>>> Cc: build-dev <[email protected]>; Doerr, Martin >>>>> <[email protected]> >>>>> Subject: RE: 8201226 missing JNIEXPORT / JNICALL at some places in >>> function >>>>> declarations/implementations - was : RE: missing JNIEXPORT / >>>>> JNICALL at some places in function declarations/implementations >>>>> >>>>>> >>>>>> Was main() exported via map files? >>>>>> >>>>> >>>>> Seems main was exported , I can find it in jdk10 in e.g. : >>>>> >>>>> make/mapfiles/launchers/mapfile-sparcv9 >>>>> make/mapfiles/launchers/mapfile-x86_64 >>>>> >>>>> >>>>> Best regards, Matthias >>>>> >>>>> >>>>>> -----Original Message----- >>>>>> From: Alexey Ivanov [mailto:[email protected]] >>>>>> Sent: Mittwoch, 11. April 2018 11:11 >>>>>> To: Baesken, Matthias <[email protected]>; Magnus Ihse >>> Bursie >>>>>> <[email protected]> >>>>>> Cc: build-dev <[email protected]>; Doerr, Martin >>>>>> <[email protected]> >>>>>> Subject: Re: 8201226 missing JNIEXPORT / JNICALL at some places in >>>>> function >>>>>> declarations/implementations - was : RE: missing JNIEXPORT / >>>>>> JNICALL at some places in function declarations/implementations >>>>>> >>>>>> >>>>>> On 11/04/2018 08:44, Baesken, Matthias wrote: >>>>>>>> JIMAGE_FindResource doesn't have JNICALL modifier now, does it? >>>>>>> Hi Alexey, yes that's true . >>>>>>> >>>>>>>> Please remove JNIEXPORT from main(): >>>>>>>> src/java.base/share/native/launcher/main.c >>>>>>>> src/jdk.pack/share/native/unpack200/main.cpp >>>>>>> I would prefer to keep it for now . >>>>>>> I notice some comments in our SAPJVM code base about needing >>>>>> JNIEXPORT for main for Solaris (we were running in SAPJVM >>>>>> without mapfiles in the past already). >>>>>>> Maybe that’s related to >>>>>>> >>>>>>> src/java.base/unix/native/libjli/java_md_solinux.c >>>>>>> >>>>>>> where main is dlsym-ed : fptr = (int (*)())dlsym(RTLD_DEFAULT, >>> "main"); >>>>>>> but I am not sure about this. >>>>>>> So I better keep the JNIEXPORT for the main functions, could be >>>>>> removed in another cleanup if really needed. >>>>>> >>>>>> OK. Let them stay then. >>>>>> Was main() exported via map files? >>>>>> >>>>>> >>>>>> The change looks good to me. >>>>>> >>>>>> Regards, >>>>>> Alexey >>>>>> >>>>>>> >>>>>>>> You can reference both yourself and me as >>>>>>>> Contributed-by: mbaesken, aivanov when pushing the changeset if >>>>>>>> you don't mind. >>>>>>>> >>>>>>> Sure . >>>>>>> >>>>>>> Best regards, Matthias >>>>>>> >>>>>>> >>>>>>>> -----Original Message----- >>>>>>>> From: Alexey Ivanov [mailto:[email protected]] >>>>>>>> Sent: Dienstag, 10. April 2018 21:34 >>>>>>>> To: Baesken, Matthias <[email protected]>; Magnus Ihse >>>>>> Bursie >>>>>>>> <[email protected]> >>>>>>>> Cc: build-dev <[email protected]>; Doerr, Martin >>>>>>>> <[email protected]> >>>>>>>> Subject: Re: 8201226 missing JNIEXPORT / JNICALL at some places >>>>>>>> in >>>>>> function >>>>>>>> declarations/implementations - was : RE: missing JNIEXPORT / >>>>>>>> JNICALL >>>>> at >>>>>>>> some places in function declarations/implementations >>>>>>>> >>>>>>>> Hi Matthias, >>>>>>>> >>>>>>>> On 10/04/2018 11:14, Baesken, Matthias wrote: >>>>>>>>> Hello, I had to do another small adjustment to make >>> jimage.hpp/cpp >>>>>> match. Please review : >>>>>>>>> >>>>>>>>> http://cr.openjdk.java.net/~mbaesken/webrevs/8201226.2/ >>>>>>>> JIMAGE_FindResource doesn't have JNICALL modifier now, does it? >>>>>>>> >>>>>>>> I've successfully built 32 bit Windows with your patch. >>>>>>>> >>>>>>>> >>>>>>>> Please remove JNIEXPORT from main(): >>>>>>>> src/java.base/share/native/launcher/main.c >>>>>>>> src/jdk.pack/share/native/unpack200/main.cpp >>>>>>>> >>>>>>>>> With the latest webrev I could finally build jdk/jdk >>>>>>>>> successfully on >>> both >>>>>> win32bit and win64 bit. >>>>>>>>> >>>>>>>>> Thanks again to Alexey to provide the incorporated patch . >>>>>>>> You can reference both yourself and me as >>>>>>>> Contributed-by: mbaesken, aivanov when pushing the changeset if >>>>>>>> you don't mind. >>>>>>>> >>>>>>>> >>>>>>>> Regards, >>>>>>>> Alexey >>>>>>>> >>>>>>>>> >>>>>>>>> Best regards, Matthias >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>>> -----Original Message----- >>>>>>>>>> From: Alexey Ivanov [mailto:[email protected]] >>>>>>>>>> Sent: Montag, 9. April 2018 17:14 >>>>>>>>>> To: Baesken, Matthias <[email protected]>; Magnus >>> Ihse >>>>>>>> Bursie >>>>>>>>>> <[email protected]> >>>>>>>>>> Cc: build-dev <[email protected]>; Doerr, Martin >>>>>>>>>> <[email protected]> >>>>>>>>>> Subject: Re: 8201226 missing JNIEXPORT / JNICALL at some >>>>>>>>>> places in >>>>>>>> function >>>>>>>>>> declarations/implementations - was : RE: missing JNIEXPORT / >>>>> JNICALL >>>>>> at >>>>>>>>>> some places in function declarations/implementations >>>>>>>>>> >>>>>>>>>> Hi Matthias, >>>>>>>>>> >>>>>>>>>> On 09/04/2018 15:38, Baesken, Matthias wrote: >>>>>>>>>>> Hi Alexey, thanks for the diff provided by you, and for the >>>>>>>> explanations >>>>>>>>>> . >>>>>>>>>>> I created a second webrev : >>>>>>>>>>> >>>>>>>>>>> http://cr.openjdk.java.net/~mbaesken/webrevs/8201226.1/ >>>>>>>>>>> >>>>>>>>>>> - it adds the diff provided by you (hope that’s fine with >>>>>>>>>>> you) >>>>>>>>>> Yes, that's fine with me. >>>>>>>>>> There could be only one author ;) >>>>>>>>>> >>>>>>>>>>> - changes 2 launchers >>>>> src/java.base/share/native/launcher/main.c >>>>>>>> and >>>>>>>>>> src/jdk.pack/share/native/unpack200/main.cpp where we face >>>>>> similar >>>>>>>>>> issues after mapfile removal for exes >>>>>>>>>> >>>>>>>>>> I'd rather remove both JNIEXPORT and JNICALL from main(). >>>>>>>>>> It wasn't exported, and it shouldn't be. >>>>>>>>>> >>>>>>>>>> Regards, >>>>>>>>>> Alexey >>>>>>>>>> >>>>>>>>>>> Best regards , Matthias >>>> >> >
