Re: [PR] RAT-77: output files with missing headers [creadur-rat]

2024-03-23 Thread via GitHub
Claudenw commented on code in PR #224: URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536599628 ## apache-rat/src/site/apt/index.apt.vm: ## @@ -79,12 +79,10 @@ usage: java -jar apache-rat/target/apache-rat-${project.version}.jar [options]

[jira] [Commented] (RAT-366) Switch to Reading preparing header text and processing in one call.

2024-03-23 Thread Claude Warren (Jira)
[ https://issues.apache.org/jira/browse/RAT-366?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830061#comment-17830061 ] Claude Warren commented on RAT-366: --- Fix in [pull 228|https://github.com/apache/creadur-rat/pull/228] >

Re: [PR] [RAT-362] Fix gitignore directory handling [creadur-rat]

2024-03-23 Thread via GitHub
nielsbasjes commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016494164 @Claudenw I think the testing issue https://github.com/apache/creadur-rat/pull/230 should be fixed first because otherwise there is no way to show this actually works on all JDK

Re: [PR] RAT-77: output files with missing headers [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on code in PR #224: URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536663580 ## apache-rat-core/src/main/java/org/apache/rat/Report.java: ## @@ -303,15 +314,13 @@ static Options buildOptions() { opts.addOption(null,

Re: [PR] RAT-77: output files with missing headers [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on code in PR #224: URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536663808 ## apache-rat/src/site/apt/index.apt.vm: ## @@ -79,12 +79,10 @@ usage: java -jar apache-rat/target/apache-rat-${project.version}.jar [options]

Re: [PR] RAT-77: output files with missing headers [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on code in PR #224: URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536663898 ## src/site/apt/index.apt.vm: ## @@ -111,6 +113,17 @@ java -jar apache-rat-${project.version}.jar --help +--+ Read

[jira] [Created] (RAT-367) Not all tests are run during the build

2024-03-23 Thread Niels Basjes (Jira)
Niels Basjes created RAT-367: Summary: Not all tests are run during the build Key: RAT-367 URL: https://issues.apache.org/jira/browse/RAT-367 Project: Apache Rat Issue Type: Bug

Re: [PR] RAT-362: Fix gitignore directory handling - switch to relative paths in underlying library [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016547369 @nielsbasjes pls rebase your branch in order for all build checks to run again (as RAT-367 is merged now). Thanks -- This is an automated message from the Apache Git Service. To

Re: [PR] [RAT-362] Fix gitignore directory handling [creadur-rat]

2024-03-23 Thread via GitHub
nielsbasjes commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016491170 For the tests I created this https://issues.apache.org/jira/browse/RAT-367 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Updated] (RAT-367) Older junit3 tests are not run during the build after migrating to junit5

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-367?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger updated RAT-367: -- Description: There are some tests that use the old Junit 3 APIs and these are not run by default

[jira] [Commented] (RAT-336) Use assertj in RAT tests

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-336?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830107#comment-17830107 ] Philipp Ottlinger commented on RAT-336: --- Maybe we could get rid of jUnit3 tests as well while

[jira] [Commented] (RAT-367) Older junit3 tests are not run during the build after migrating to junit5

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830108#comment-17830108 ] ASF subversion and git services commented on RAT-367: - Commit

Re: [PR] RAT-367: Enable running old junit tests during the build with junit5 [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on PR #230: URL: https://github.com/apache/creadur-rat/pull/230#issuecomment-2016545480 Thanks for the catch, I'll add a changelog entry accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] RAT-367: Enable running old junit tests during the build with junit5 [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger merged PR #230: URL: https://github.com/apache/creadur-rat/pull/230 -- 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:

[jira] [Updated] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger updated RAT-362: -- Summary: plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

[jira] [Assigned] (RAT-362) plugin: .gitignore not correctly applied

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger reassigned RAT-362: - Assignee: Philipp Ottlinger (was: Jean-Baptiste Onofré) > plugin: .gitignore not

[jira] [Updated] (RAT-345) Dependency updates via dependabot performed for release 0.17

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-345?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger updated RAT-345: -- Description: h1. Updates via dependabot in RAT 0.17 h2. (/) Update Bump assertj-core from 3.25.1

[jira] [Updated] (RAT-367) Older junit3 tests are not run during the build after migrating to junit5

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-367?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger updated RAT-367: -- Component/s: build Fix Version/s: 0.17 Affects Version/s: 0.16.1 > Older junit3

[jira] [Commented] (RAT-367) Older junit3 tests are not run during the build after migrating to junit5

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830111#comment-17830111 ] ASF subversion and git services commented on RAT-367: - Commit

[jira] [Resolved] (RAT-367) Older junit3 tests are not run during the build after migrating to junit5

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-367?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger resolved RAT-367. --- Resolution: Fixed Thanks > Older junit3 tests are not run during the build after migrating to

Re: [PR] RAT-362, RAT-345: Fix gitignore directory handling - switch to relative paths in underlying library [creadur-rat]

2024-03-23 Thread via GitHub
nielsbasjes commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016552089 I suspect the new test will fail on Windows because I expect to get a drive letter in the middle. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] RAT-362, RAT-345: Fix gitignore directory handling - switch to relative paths in underlying library [creadur-rat]

2024-03-23 Thread via GitHub
nielsbasjes commented on code in PR #229: URL: https://github.com/apache/creadur-rat/pull/229#discussion_r1536664496 ## apache-rat-plugin/src/test/resources/unit/RAT-362-GitIgnore/pom.xml: ## @@ -0,0 +1,43 @@ + + +http://maven.apache.org/POM/4.0.0;

[jira] [Commented] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830169#comment-17830169 ] ASF subversion and git services commented on RAT-362: - Commit

Re: [PR] RAT-362, RAT-345: Fix gitignore directory handling - switch to relative paths in underlying library [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger merged PR #229: URL: https://github.com/apache/creadur-rat/pull/229 -- 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:

[jira] [Commented] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830171#comment-17830171 ] ASF subversion and git services commented on RAT-362: - Commit

[jira] [Commented] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830168#comment-17830168 ] ASF subversion and git services commented on RAT-362: - Commit

[jira] [Commented] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830170#comment-17830170 ] ASF subversion and git services commented on RAT-362: - Commit

[jira] [Commented] (RAT-345) Dependency updates via dependabot performed for release 0.17

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-345?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830172#comment-17830172 ] ASF subversion and git services commented on RAT-345: - Commit

Re: [PR] RAT-333: fix --force option will change the file permissions [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on PR #223: URL: https://github.com/apache/creadur-rat/pull/223#issuecomment-2016621846 Thanks for spotting this error and providing a fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[jira] [Commented] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830175#comment-17830175 ] ASF subversion and git services commented on RAT-362: - Commit

[jira] [Commented] (RAT-345) Dependency updates via dependabot performed for release 0.17

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-345?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830176#comment-17830176 ] ASF subversion and git services commented on RAT-345: - Commit

Re: [PR] RAT-77: output files with missing headers [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on code in PR #224: URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536663580 ## apache-rat-core/src/main/java/org/apache/rat/Report.java: ## @@ -303,15 +314,13 @@ static Options buildOptions() { opts.addOption(null,

[jira] [Updated] (RAT-77) output missing headers inline from plugin

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-77?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger updated RAT-77: - Fix Version/s: 0.17 > output missing headers inline from plugin >

[jira] [Commented] (RAT-345) Dependency updates via dependabot performed for release 0.17

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-345?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830181#comment-17830181 ] ASF subversion and git services commented on RAT-345: - Commit

Re: [PR] RAT-333: fix --force option will change the file permissions [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger merged PR #223: URL: https://github.com/apache/creadur-rat/pull/223 -- 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:

[jira] [Commented] (RAT-333) --force option will change the file permissions

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830179#comment-17830179 ] ASF subversion and git services commented on RAT-333: - Commit

[jira] [Commented] (RAT-333) --force option will change the file permissions

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830180#comment-17830180 ] ASF subversion and git services commented on RAT-333: - Commit

[jira] [Resolved] (RAT-333) --force option will change the file permissions

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-333?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger resolved RAT-333. --- Resolution: Fixed Merged. > --force option will change the file permissions >

Re: [PR] RAT-362, RAT-345: Fix gitignore directory handling - switch to relative paths in underlying library [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016612423 @Claudenw I do hope it is fine with 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

[jira] [Resolved] (RAT-362) plugin: .gitignore not correctly applied - bugfix in underlying gitignore-reader library

2024-03-23 Thread Philipp Ottlinger (Jira)
[ https://issues.apache.org/jira/browse/RAT-362?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philipp Ottlinger resolved RAT-362. --- Resolution: Fixed [~engelen] would you mind giving feedback if the current 0.17-SNAPSHOT fixes

Re: [PR] RAT-77: output files with missing headers [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on code in PR #224: URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536701128 ## src/site/apt/index.apt.vm: ## @@ -111,6 +113,17 @@ java -jar apache-rat-${project.version}.jar --help +--+ Read

Re: [PR] [RAT-362] Fix gitignore directory handling [creadur-rat]

2024-03-23 Thread via GitHub
nielsbasjes commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016485876 > @nielsbasjes do you have a test that shows failure before this fix is applied? Yes. In this case it is a problem in the upstream library (which I wrote) where I have

Re: [PR] [RAT-362] Fix gitignore directory handling [creadur-rat]

2024-03-23 Thread via GitHub
nielsbasjes commented on PR #229: URL: https://github.com/apache/creadur-rat/pull/229#issuecomment-2016492427 Also a merge request to fix this testing issue https://github.com/apache/creadur-rat/pull/230 -- This is an automated message from the Apache Git Service. To respond to the

[jira] [Commented] (RAT-367) Older junit3 tests are not run during the build after migrating to junit5

2024-03-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/RAT-367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17830109#comment-17830109 ] ASF subversion and git services commented on RAT-367: - Commit

Re: [PR] RAT-362: Fix gitignore directory handling - switch to relative paths in underlying library [creadur-rat]

2024-03-23 Thread via GitHub
ottlinger commented on code in PR #229: URL: https://github.com/apache/creadur-rat/pull/229#discussion_r1536662289 ## apache-rat-plugin/src/test/resources/unit/RAT-362-GitIgnore/pom.xml: ## @@ -0,0 +1,43 @@ + + +http://maven.apache.org/POM/4.0.0;