Hello Patrick, try reverting common/autoconf/generated-configure before generating the patch. That file does not need to be reviewed as it's just generated anyway. That should reduce the patch size.

/Erik

On 2014-11-14 09:54, Patrick Reinhart wrote:
Hi Magnus

Unfortunately the patch itself is as big as 2 megabytes and will therefore get 
removed… Now I’m really stuck.

Cheers Patrick

Am 14.11.2014 um 09:45 schrieb Magnus Ihse Bursie 
<magnus.ihse.bur...@oracle.com>:

Hi Patrick,

I'm sorry for all the trouble you have encountered when trying to help 
improving OpenJDK. :(

Try sending the patch only (not the webrev), as normal inlined text (not an 
attachment) in a mail to the list. I can create a webrev and put it on 
cr.openjdk.java.net <http://cr.openjdk.java.net/> for you.

/Magnus

14 nov 2014 kl. 09:24 skrev Patrick Reinhart <patr...@reini.net 
<mailto:patr...@reini.net>>:

Hi David,

I’ve got a mail that the attachment was too big and was held by the moderator 
approval. I personally would love to put that on cr.openjdk.java.net 
<http://cr.openjdk.java.net/> but to put it there it seems that I fist has to 
be contributor and in order to be that I had be submit some patches first :-(

It seems I’m stuck in a recursive loop somehow ;-)


Cheers Patrick

PS: I did already contributed two fixes for the 2d printing part in the past.


Am 14.11.2014 um 09:01 schrieb David Holmes <david.hol...@oracle.com 
<mailto:david.hol...@oracle.com>>:

On 14/11/2014 5:36 PM, Patrick Reinhart wrote:
An attachment in the email has been found to contain executable code and has 
been removed.

File removed : webrev.zip, zip,html,js
Unclear what removed the attachment. :(

David

----------------------------------------------------------------------
Attached the webrev





Am 14.11.2014 um 03:18 schrieb David Holmes <david.hol...@oracle.com 
<mailto:david.hol...@oracle.com>>:

Hi Patrick,

On 14/11/2014 12:15 AM, Patrick Reinhart wrote:
Hi Magnus,

I actually merged down the freetype compilation from the source during the 
configuration cycle as it’s being done in JDK9 actually. This was not that 
straightforward but I could sort that out and had a successful compilation 
afterwards.

Here is the link to the webrev:  http://reinharts.dyndns.org/buildjdk8win/webrev/ 
<http://reinharts.dyndns.org/buildjdk8win/webrev/> 
<http://reinharts.dyndns.org/buildjdk8win/webrev/ 
<http://reinharts.dyndns.org/buildjdk8win/webrev/>>
Please note that patches have to be submitted via OpenJDK infrastructure: 
cr.openjdk.java.net <http://cr.openjdk.java.net/> or email attachment to 
OpenJDK mailing list.

Cheers,
David

[1] http://openjdk.java.net/legal/tou/ <http://openjdk.java.net/legal/tou/>




Cheers

Patrick


Am 13.11.2014 um 14:50 schrieb Magnus Ihse Bursie <magnus.ihse.bur...@oracle.com 
<mailto:magnus.ihse.bur...@oracle.com>>:

On 2014-11-12 19:34, Patrick Reinhart wrote:
Hi there,

I’m updating the Adopt OpenJDK website with build instructions to build the 
OpenJDK on Windows 8.1. In order to lower the effort of building also the 
OpenJDK 8 on Windows, myself and some colleges have back ported the build 
scripts from OpenJDK 9. We also could provide a webrev containing all changes 
we made.

Could someone sponsor this back port?
Sure, I can sponsor that!

Is the patch straightforward to apply on jdk9 from jdk8?

/Magnus

Reply via email to