You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@netbeans.apache.org by "ebarboni (via GitHub)" <gi...@apache.org> on 2026/09/24 16:51:07 UTC

[PR] upgrade platform lib [netbeans]

ebarboni opened a new pull request, #9641:
URL: https://github.com/apache/netbeans/pull/9641

   upgrade libraries in platform cluster.
   
   commons (io,lang3,logging,codec)
   
   jsvg
   
   upgrade the test framework update testNG 7.12.0 and junit (only to 5.14.4 not major but looks more easy thant 4 to 5 migration).
   
   maven project generation use same version now
   ant projet testng lib synchronized with 7.12.0
   
   may help fixing: #9561 integrated the modification suggested by @matthiasblaesing 
   
   Used a new bundle to progress indicator as there is a bundle error, not sure the root cause java/testng/src/org/netbeans/modules/testng/ProgressIndicator.java
   


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade some commons libs, jsvg, junit and testng [netbeans]

Posted by "mbien (via GitHub)" <gi...@apache.org>.
mbien commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5872091989

   jumbo lib update PRs are probably the only practical way to do this tbh. Risky updates which need separate testing do deserve their own PR, but the manual labor which is required to maintain a repo like this one without dependabot is somewhat reduced by bundling dependencies.
   
   Not to mention that the committer has to play maven resolver and look at the tree.


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade platform lib [netbeans]

Posted by "ebarboni (via GitHub)" <gi...@apache.org>.
ebarboni commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5828810931

   @mbien done,


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade some commons libs, jsvg, junit and testng [netbeans]

Posted by "ebarboni (via GitHub)" <gi...@apache.org>.
ebarboni merged PR #9641:
URL: https://github.com/apache/netbeans/pull/9641


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade some commons libs, jsvg, junit and testng [netbeans]

Posted by "neilcsmith-net (via GitHub)" <gi...@apache.org>.
neilcsmith-net commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5871550850

   One downside of using a single PR will be the release notes generation. Not a reason not to do it, but we might want to edit the generated notes where third-party deps are concerned.


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade some commons libs, jsvg, junit and testng [netbeans]

Posted by "mbien (via GitHub)" <gi...@apache.org>.
mbien commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5871412316

   > no squash I suppose for this one
   
   right, I think it is better to merge it as multiple commits. Makes reverts easier if something happens.


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade platform lib [netbeans]

Posted by "ebarboni (via GitHub)" <gi...@apache.org>.
ebarboni commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5832655014

   As some check where in red I look a bit the log.
   
   looks like compiler.flags entries in the test generated lots of warning 
   WARNING: Unknown module: jdk.compiler specified to --add-exports
   
   Maybe continuing the ordering of some items could reduce the 
   WARNING: Not all children in / marked with the position attribute: with lots of items inside


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade platform lib [netbeans]

Posted by "mbien (via GitHub)" <gi...@apache.org>.
mbien commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5818756567

   IMO: multi lib updates (of mostly unrelated libraries) should be split into individual commits, one per updated lib wrapper. Each commit should also get a proper commit message. "upgrade platform lib" for so many libraries is not sufficient to tell the story what the commit actually includes.
   
   This could be still fixed here since wrapper updates are usually folder based. e.g `git reset HEAD^` + `for-each-lib: {git add <lib wrapper> + git commit }`. Some adjustment might be needed so that everything what belongs together should is in one commit, e.g test ng lib update commit should include all changes required to make that lib update work.


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists


Re: [PR] upgrade some commons libs, jsvg, junit and testng [netbeans]

Posted by "ebarboni (via GitHub)" <gi...@apache.org>.
ebarboni commented on PR #9641:
URL: https://github.com/apache/netbeans/pull/9641#issuecomment-5868387659

   no squash I suppose for this one


-- 
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-mail: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists