Build failed in Jenkins: POI » POI-DSL-saxon-test #68

2023-02-04 Thread Apache Jenkins Server
See Changes: [PJ Fanning] fix compile issues [PJ Fanning] [bug-65260] catch Throwable instead of error - see https://github.com/apache/poi/pull/425 [PJ Fanning] mark some variables that are used in

Build failed in Jenkins: POI » POI-DSL-Windows-1.18 #65

2023-02-04 Thread Apache Jenkins Server
See Changes: [PJ Fanning] fix compile issues [PJ Fanning] [bug-65260] catch Throwable instead of error - see https://github.com/apache/poi/pull/425 [PJ Fanning] mark some variables that are used i

Build failed in Jenkins: POI » POI-DSL-1.8 #835

2023-02-04 Thread Apache Jenkins Server
See Changes: [PJ Fanning] fix compile issues [PJ Fanning] [bug-65260] catch Throwable instead of error - see https://github.com/apache/poi/pull/425 -- [...truncate

[GitHub] [poi] xzel23 commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
xzel23 commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416840963 Wouldn't it be better to guard awt calls by checking for headless mode instead of catching UnsatisfiedLinkError? -- This is an automated message from the Apache Git Service. To respond to the

[Bug 66436] After the .xlsx file is encrypted by poi and then decrypted by POI, an error will be reported when it is opened with Microsoft Excel, and an error will also be reported when it is decompre

2023-02-04 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66436 --- Comment #18 from Dominik Stadler --- I could narrow this down to the following test-case and the following findings: * The length of the byte[] is related, e.g. using 4000 bytes works, but 3292 like here fails * The actual byte-data seems

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416785554 The problem is that AWT is not available in a GraalVM native image built in MacOS because of https://github.com/oracle/graal/issues/4124, so I get `NoClassDefFoundError`s and the `java.lang.Un

[GitHub] [poi] pjfanning commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416784344 Can you provide a stack trace where getFontAt fails? If that is failing, I think there will be loads of changes needed. Would it not be easier to install fonts yourself? -- This is a

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416783091 > So I updated SheetUtil in [040181c](https://github.com/apache/poi/commit/040181c42fdfbe15b706e8ae81f5e2f6fe87554c) - is that enough? I think you also need to add catch any AWT operati

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416782703 This is the stacktrace when the `org.apache.poi.ss.ignoreMissingFontSystem` is set to `true`: ```java Caused by: java.lang.NoClassDefFoundError: Could not initialize class java.awt.

[GitHub] [poi] pjfanning commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416782615 So I updated SheetUtil in https://github.com/apache/poi/commit/040181c42fdfbe15b706e8ae81f5e2f6fe87554c - is that enough? -- This is an automated message from the Apache Git Service. To re

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416782378 > Thanks - the exception name is enough for me - so UnsatisfiedLinkError is the issue Yes, as I mentioned in the first comment of this PR 😄 -- This is an automated message from the A

[GitHub] [poi] pjfanning commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416782228 Thanks - the exception name is enough for me - so UnsatisfiedLinkError is the issue -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416782086 Here is the stacktrace without the system property set: ``` Caused by: java.lang.UnsatisfiedLinkError: no awt in java.library.path at org.graalvm.nativeimage.builder/com.orac

[GitHub] [poi] gastaldi closed pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi closed pull request #425: Catch `Throwable` when using the AWT classes URL: https://github.com/apache/poi/pull/425 -- 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. To u

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416781744 > The catch you modified - that was only added recently and no release has ever included it. Are you sure that InternalError catch was not enough? Yes, I am sure. The thrown exception is

[GitHub] [poi] pjfanning commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416781069 Can you share the stack traces you are getting? -- 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

[GitHub] [poi] gastaldi commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416780928 Thanks, I noticed that `SheetUtil` also uses some AWT code in `org.apache.poi.ss.util.SheetUtil.getCellWidth` that could use this try-catch as well -- This is an automated message from the

[GitHub] [poi] pjfanning commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416780526 The catch you modified - that was only added recently and no release has ever included it. Are you sure that InternalError catch was not enough? -- This is an automated message from the Apa

[Bug 65260] Creating an SXSSFWorkbook fails if no fonts are available on the system

2023-02-04 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65260 --- Comment #5 from PJ Fanning --- I added r1907308 due to https://github.com/apache/poi/pull/425 - but looks like it might be causing issues itself -- You are receiving this mail because: You are the assignee for the bug. ---

[GitHub] [poi] pjfanning commented on pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on PR #425: URL: https://github.com/apache/poi/pull/425#issuecomment-1416778735 Can you close this? - I committed https://github.com/apache/poi/commit/040181c42fdfbe15b706e8ae81f5e2f6fe87554c and there are a few more similar cases in the code that should probably be cha

[GitHub] [poi] dependabot[bot] commented on pull request #424: Bump org.mockito:mockito-core from 4.11.0 to 5.1.1

2023-02-04 Thread via GitHub
dependabot[bot] commented on PR #424: URL: https://github.com/apache/poi/pull/424#issuecomment-1416777934 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me kn

[GitHub] [poi] asfgit closed pull request #424: Bump org.mockito:mockito-core from 4.11.0 to 5.1.1

2023-02-04 Thread via GitHub
asfgit closed pull request #424: Bump org.mockito:mockito-core from 4.11.0 to 5.1.1 URL: https://github.com/apache/poi/pull/424 -- 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.

[GitHub] [poi] pjfanning commented on a diff in pull request #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
pjfanning commented on code in PR #425: URL: https://github.com/apache/poi/pull/425#discussion_r1096547662 ## poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFSheet.java: ## @@ -63,7 +63,7 @@ protected SXSSFSheet(SXSSFWorkbook workbook, XSSFSheet xSheet, int randomAcc

[GitHub] [poi] gastaldi opened a new pull request, #425: Catch `Throwable` when using the AWT classes

2023-02-04 Thread via GitHub
gastaldi opened a new pull request, #425: URL: https://github.com/apache/poi/pull/425 `ERROR: java.lang.UnsatisfiedLinkError: no awt in java.library.path` is thrown when attempting to build a native image that depends on AWT on macOS. See the following issue for more details: -