Build failed in Jenkins: POI » POI-DSL-OpenJDK #1023

2024-11-19 Thread Apache Jenkins Server
See Changes: [PJ Fanning] [bug-66590] Number of blocks used by the property table missing from the file header. Thanks to Emmanuel Bourg. This closes #728 -- S

Re: [DISCUSS] working around log4j issues

2024-11-19 Thread PJ Fanning
I've added a failover wrapper for LogManager in XMLBeans. There are a couple of mistakes here - but that have been modified since: https://github.com/apache/xmlbeans/commit/89fc0e4e8bd0a95f14ada7afe6901932d3172356 I would suggest that you use this commit as an idea of the change but check the la

Re: [PR] Write the number of blocks used by the property table in the header of files using 4K blocks only [poi]

2024-11-19 Thread via GitHub
ebourg commented on PR #728: URL: https://github.com/apache/poi/pull/728#issuecomment-2486448136 Thank you -- 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 unsubscribe, e

[Bug 69463] New: CellUtil.setCellStyleProperties sets the cell background color to black

2024-11-19 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69463 Bug ID: 69463 Summary: CellUtil.setCellStyleProperties sets the cell background color to black Product: POI Version: 5.3.0-FINAL Hardware: Macintosh OS: o

[Bug 69463] CellUtil.setCellStyleProperties sets the cell background color to black

2024-11-19 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69463 --- Comment #1 from Donal --- Created attachment 39940 --> https://bz.apache.org/bugzilla/attachment.cgi?id=39940&action=edit Input file for test cases Save this file in /tmp before running the tests. If you want to use a different directory

[Bug 66590] Number of blocks used by the property table missing from the file header

2024-11-19 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66590 --- Comment #5 from PJ Fanning --- https://github.com/apache/poi/pull/728 says: The header field must be updated only if the file uses 4K blocks. If small blocks are used the field must remain nul. -- You are receiving this mail because: You

Re: [PR] Bump org.sonarqube from 4.0.0.2929 to 6.0.0.5145 [poi]

2024-11-19 Thread via GitHub
asfgit closed pull request #729: Bump org.sonarqube from 4.0.0.2929 to 6.0.0.5145 URL: https://github.com/apache/poi/pull/729 -- 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. T

Re: [PR] Write the number of blocks used by the property table in the header of files using 4K blocks only [poi]

2024-11-19 Thread via GitHub
asfgit closed pull request #728: Write the number of blocks used by the property table in the header of files using 4K blocks only URL: https://github.com/apache/poi/pull/728 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] Bump org.sonarqube from 4.0.0.2929 to 6.0.0.5145 [poi]

2024-11-19 Thread via GitHub
dependabot[bot] commented on PR #729: URL: https://github.com/apache/poi/pull/729#issuecomment-2486351541 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

[PR] Bump org.sonarqube from 4.0.0.2929 to 6.0.0.5145 [poi]

2024-11-19 Thread via GitHub
dependabot[bot] opened a new pull request, #729: URL: https://github.com/apache/poi/pull/729 Bumps org.sonarqube from 4.0.0.2929 to 6.0.0.5145. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.sonarqube&packa

Re: [PR] Write the number of blocks used by the property table in the header of files using 4K blocks only [poi]

2024-11-19 Thread via GitHub
ebourg commented on PR #728: URL: https://github.com/apache/poi/pull/728#issuecomment-2485835355 Fixed, the tests pass now. -- 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.

Re: [PR] Write the number of blocks used by the property table in the header of files using 4K blocks only [poi]

2024-11-19 Thread via GitHub
ebourg commented on PR #728: URL: https://github.com/apache/poi/pull/728#issuecomment-2485750799 Indeed, I misused writeOutAndReadBack, the tested files are modified but shouldn't. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [DISCUSS] working around log4j issues

2024-11-19 Thread Tim Allison
Got it. Makes sense. Thank you. On Sat, Nov 16, 2024 at 2:35 PM PJ Fanning wrote: > > Hi Tim, > I stand by what I said earlier in the thread: that it is counterproductive to > make yet another change to our logging framework - even if it is a return to > one we used before. slf4j-api is not imm

Re: [PR] Write the number of blocks used by the property table in the header of files using 4K blocks only [poi]

2024-11-19 Thread via GitHub
pjfanning commented on PR #728: URL: https://github.com/apache/poi/pull/728#issuecomment-2485589143 This causes a lot of test failures. Example: Test testWritePropertyCount4096() FAILED -- This is an automated message from the Apache Git Service. To respond to the message, please

Build failed in Jenkins: POI » POI-DSL-OpenJDK #1022

2024-11-19 Thread Apache Jenkins Server
See Changes: [PJ Fanning] [gitub-727] Remove DSTAMP token substitution. Thanks to Emmanuel Bourg. This closes #727 -- [...truncated 211 B...] [EnvInject] - Pre

Re: [PR] Do not replace the @DSTAMP@ token when generating Version.java [poi]

2024-11-19 Thread via GitHub
ebourg commented on PR #727: URL: https://github.com/apache/poi/pull/727#issuecomment-2485227636 Thanks! -- 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 unsubscribe, e-m

Re: [PR] Do not replace the @DSTAMP@ token when generating Version.java [poi]

2024-11-19 Thread via GitHub
asfgit closed pull request #727: Do not replace the @DSTAMP@ token when generating Version.java URL: https://github.com/apache/poi/pull/727 -- 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 specif

[PR] Do not replace the @DSTAMP@ token when generating Version.java [poi]

2024-11-19 Thread via GitHub
ebourg opened a new pull request, #727: URL: https://github.com/apache/poi/pull/727 The timestamp was removed from `Version.java` last year (23caf67c58fb5e6ecbe50eb1a24b02ef40585629), but the Ant and Gradle builds still attempt to replace the `@DSTAMP@` token. This PR removes this now unnec