Re: [PR] Bump org.apache.maven.plugins:maven-shade-plugin from 3.5.2 to 3.5.3 [creadur-whisker]

2024-04-23 Thread via GitHub


ottlinger merged PR #141:
URL: https://github.com/apache/creadur-whisker/pull/141


-- 
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: dev-unsubscr...@creadur.apache.org

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



[PR] Bump org.apache.maven.plugins:maven-shade-plugin from 3.5.2 to 3.5.3 [creadur-whisker]

2024-04-23 Thread via GitHub


dependabot[bot] opened a new pull request, #141:
URL: https://github.com/apache/creadur-whisker/pull/141

   Bumps 
[org.apache.maven.plugins:maven-shade-plugin](https://github.com/apache/maven-shade-plugin)
 from 3.5.2 to 3.5.3.
   
   Commits
   
   https://github.com/apache/maven-shade-plugin/commit/19618cde0df300251dfd49bb5a043e31b8dfe8c0;>19618cd
 [maven-release-plugin] prepare release maven-shade-plugin-3.5.3
   https://github.com/apache/maven-shade-plugin/commit/faf233e315c7d8876a9714e15f3427429de0f9dd;>faf233e
 [MSHADE-472] upgrade parent POM
   https://github.com/apache/maven-shade-plugin/commit/7de9ae718876c5b1a9808726c8ad1e101aa5db0d;>7de9ae7
 [MSHADE-471] deal with DST
   https://github.com/apache/maven-shade-plugin/commit/58d8cfe097f995a944c922552afcc14e68eec51b;>58d8cfe
 [MSHADE-470] Bump asmVersion from 9.6 to 9.7 (https://redirect.github.com/apache/maven-shade-plugin/issues/218;>#218)
   https://github.com/apache/maven-shade-plugin/commit/90745827b50e1323674e5bc7c1ccdeeddf87afd1;>9074582
 Bump org.apache.commons:commons-compress from 1.26.0 to 1.26.1
   https://github.com/apache/maven-shade-plugin/commit/a8030f829180f50d4497e42e212a891ec595a059;>a8030f8
 Bump org.apache.commons:commons-compress from 1.25.0 to 1.26.0 (https://redirect.github.com/apache/maven-shade-plugin/issues/215;>#215)
   https://github.com/apache/maven-shade-plugin/commit/1218bdcc8d2d2d8c94c46a771efb66ba1f209eee;>1218bdc
 [maven-release-plugin] prepare for next development iteration
   See full diff in https://github.com/apache/maven-shade-plugin/compare/maven-shade-plugin-3.5.2...maven-shade-plugin-3.5.3;>compare
 view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.maven.plugins:maven-shade-plugin=maven=3.5.2=3.5.3)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot show  ignore conditions` will show all of 
the ignore conditions of the specified dependency
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   
   
   


-- 
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: dev-unsubscr...@creadur.apache.org

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



Re: [PR] Tika based document analyzer - DO NOT MERGE [creadur-rat]

2024-04-23 Thread via GitHub


ottlinger commented on PR #240:
URL: https://github.com/apache/creadur-rat/pull/240#issuecomment-2072736075

   I'd prefer a file filter that allows ignoring no-comment plaintext files 
such as JSON.


-- 
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: dev-unsubscr...@creadur.apache.org

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



Re: [PR] RAT-366: switch to single match call [creadur-rat]

2024-04-23 Thread via GitHub


Claudenw closed pull request #228: RAT-366: switch to single match call
URL: https://github.com/apache/creadur-rat/pull/228


-- 
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: dev-unsubscr...@creadur.apache.org

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



Re: [PR] Draft: RAT-355: Use annotations to build Description of Components (Matchers & Licenses) and write configuration. [creadur-rat]

2024-04-23 Thread via GitHub


Claudenw closed pull request #227: Draft: RAT-355: Use annotations to build 
Description of Components (Matchers & Licenses) and  write configuration.
URL: https://github.com/apache/creadur-rat/pull/227


-- 
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: dev-unsubscr...@creadur.apache.org

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



Re: [PR] DO NOT MERGE [creadur-rat]

2024-04-23 Thread via GitHub


Claudenw closed pull request #232: DO NOT MERGE
URL: https://github.com/apache/creadur-rat/pull/232


-- 
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: dev-unsubscr...@creadur.apache.org

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



Re: [PR] RAT-355, RAT-366: Fixed closed output issue [creadur-rat]

2024-04-23 Thread via GitHub


ottlinger commented on PR #242:
URL: https://github.com/apache/creadur-rat/pull/242#issuecomment-2072711437

   Confirmed locally - the build looks okay again.


-- 
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: dev-unsubscr...@creadur.apache.org

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



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

2024-04-23 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/RAT-366?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17840137#comment-17840137
 ] 

ASF subversion and git services commented on RAT-366:
-

Commit e5c95ed53f209c6c30e512d251500527d9535586 in creadur-rat's branch 
refs/heads/master from P. Ottlinger
[ https://gitbox.apache.org/repos/asf?p=creadur-rat.git;h=e5c95ed5 ]

Merge pull request #242 from Claudenw/fix_closed_output

RAT-355, RAT-366: Fixed closed output issue

> Switch to Reading preparing header text and processing in one call.
> ---
>
> Key: RAT-366
> URL: https://issues.apache.org/jira/browse/RAT-366
> Project: Apache Rat
>  Issue Type: Improvement
>  Components: core engine
>Affects Versions: 0.16.1
>Reporter: Claude Warren
>Assignee: Claude Warren
>Priority: Minor
> Fix For: 0.17
>
>
> Currently the system spends a fair chunk of time processing input line by 
> line.  The code path is much simplified if instead we read the maximum number 
> of lines that we would process under the old code and process that all at 
> once. 
> This will also improve memory footprint since the full text matcher will not 
> have to keep a copy of all the text it has seen so far, but rather just 
> process the block of text that is passed in.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[RAT][DISCUSS] json now shows as uknown license

2024-04-23 Thread Claude Warren
I have a pull request to switch to the Tika parser [1] one of changes that
this imposes is that Tika correctly determines that json files are
plain/text type.  So they get processed for licenses and the change will
end up listing all json files as having unknown licenses.  This is a
radical breaking change from 0.16.1 where we within the guessers skipped
the json types.

We have 2 choices

   1. Add extra filtering to convert json back to the "binary" type that
   0.16.1 uses.  I think that this is an incorrect choice.
   2. Create a default file filter that we will always use.  This filter
   will eventually identify text files that, like json, do not have the
   ability to store comments.


Thoughts?

[1] https://github.com/apache/creadur-rat/pull/240


[jira] [Commented] (RAT-355) optionally Export XML configuration file as part of run.

2024-04-23 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/RAT-355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17840136#comment-17840136
 ] 

ASF subversion and git services commented on RAT-355:
-

Commit e5c95ed53f209c6c30e512d251500527d9535586 in creadur-rat's branch 
refs/heads/master from P. Ottlinger
[ https://gitbox.apache.org/repos/asf?p=creadur-rat.git;h=e5c95ed5 ]

Merge pull request #242 from Claudenw/fix_closed_output

RAT-355, RAT-366: Fixed closed output issue

> optionally Export XML configuration file as part of run.
> 
>
> Key: RAT-355
> URL: https://issues.apache.org/jira/browse/RAT-355
> Project: Apache Rat
>  Issue Type: Improvement
>  Components: core engine
>Affects Versions: 0.16
>Reporter: Claude Warren
>Assignee: Claude Warren
>Priority: Major
> Fix For: 0.17
>
>
> Currently multiples inputs are processed into the internal configuration.  
> This is request is to export the configuration as part of the processing of 
> the files.  This will be an optional output into the XML result of the run. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] RAT-355, RAT-366: Fixed closed output issue [creadur-rat]

2024-04-23 Thread via GitHub


ottlinger merged PR #242:
URL: https://github.com/apache/creadur-rat/pull/242


-- 
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: dev-unsubscr...@creadur.apache.org

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



[PR] Fixed closed output issue [creadur-rat]

2024-04-23 Thread via GitHub


Claudenw opened a new pull request, #242:
URL: https://github.com/apache/creadur-rat/pull/242

   This fixes an issue detected in the build output where the tail end of the 
output disappears, though the build appears to work correctly.
   
   The issue is that the ant report test manages to close System.out.  This 
change ensures that System.out does not get closed.
   


-- 
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: dev-unsubscr...@creadur.apache.org

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



[jira] [Commented] (RAT-355) optionally Export XML configuration file as part of run.

2024-04-23 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/RAT-355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17840034#comment-17840034
 ] 

ASF subversion and git services commented on RAT-355:
-

Commit 66e134635aa893c291df0b2b4c6e134d37b930c4 in creadur-rat's branch 
refs/heads/master from Claude Warren
[ https://gitbox.apache.org/repos/asf?p=creadur-rat.git;h=66e13463 ]

Merge pull request #233 from Claudenw/RAT-355_RAT-366_merge

RAT-355 and RAT-366 fixes in one package

> optionally Export XML configuration file as part of run.
> 
>
> Key: RAT-355
> URL: https://issues.apache.org/jira/browse/RAT-355
> Project: Apache Rat
>  Issue Type: Improvement
>  Components: core engine
>Affects Versions: 0.16
>Reporter: Claude Warren
>Assignee: Claude Warren
>Priority: Major
> Fix For: 0.17
>
>
> Currently multiples inputs are processed into the internal configuration.  
> This is request is to export the configuration as part of the processing of 
> the files.  This will be an optional output into the XML result of the run. 
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


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

2024-04-23 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/RAT-366?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17840035#comment-17840035
 ] 

ASF subversion and git services commented on RAT-366:
-

Commit 66e134635aa893c291df0b2b4c6e134d37b930c4 in creadur-rat's branch 
refs/heads/master from Claude Warren
[ https://gitbox.apache.org/repos/asf?p=creadur-rat.git;h=66e13463 ]

Merge pull request #233 from Claudenw/RAT-355_RAT-366_merge

RAT-355 and RAT-366 fixes in one package

> Switch to Reading preparing header text and processing in one call.
> ---
>
> Key: RAT-366
> URL: https://issues.apache.org/jira/browse/RAT-366
> Project: Apache Rat
>  Issue Type: Improvement
>  Components: core engine
>Affects Versions: 0.16.1
>Reporter: Claude Warren
>Assignee: Claude Warren
>Priority: Minor
> Fix For: 0.17
>
>
> Currently the system spends a fair chunk of time processing input line by 
> line.  The code path is much simplified if instead we read the maximum number 
> of lines that we would process under the old code and process that all at 
> once. 
> This will also improve memory footprint since the full text matcher will not 
> have to keep a copy of all the text it has seen so far, but rather just 
> process the block of text that is passed in.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] RAT-355 and RAT-366 fixes in one package [creadur-rat]

2024-04-23 Thread via GitHub


Claudenw merged PR #233:
URL: https://github.com/apache/creadur-rat/pull/233


-- 
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: dev-unsubscr...@creadur.apache.org

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