Title: Ignore blank and null entries in PATH search for mvn/gradle execs by scottkurz · Pull Request #127 · codellm-devkit/codeanalyzer-java · GitHub
Open Graph Title: Ignore blank and null entries in PATH search for mvn/gradle execs by scottkurz · Pull Request #127 · codellm-devkit/codeanalyzer-java
X Title: Ignore blank and null entries in PATH search for mvn/gradle execs by scottkurz · Pull Request #127 · codellm-devkit/codeanalyzer-java
Description: Motivation and Context With a null PATH entry on Windows finding the mvn command resolves to absolute path: "C:\mvn.cmd" which does not exist for me, so the dependency lib download fails. Error log msgs like: �[36m2025-02-25T10:27:24.866896�[0m�[31m [ERROR] �[0mCould not verify that C:\mvn.cmd exists �[36m2025-02-25T10:27:24.932447400�[0m�[33m [WARN] �[0mFailed to download library dependencies of project I noticed this on v1.1.0 but looking at the code it appears to exist in the main branch as well. How Has This Been Tested? Tested with an empty PATH entry on Windows Breaking Changes No Types of changes Bug fix (non-breaking change which fixes an issue) Checklist Haven't tested Additional context I did a bit of searching to try to understand if an empty PATH entry should be considered meaningful, and found some leads suggesting it should mean the current directory. I didn't follow up on this though. We wouldn't want to use map(File::getAbsolutePath) if we wanted to find something in the current directory, so that would require a different solution. It might still be better to filter this case out until/unless it is clear we want to include it.
Open Graph Description: Motivation and Context With a null PATH entry on Windows finding the mvn command resolves to absolute path: "C:\mvn.cmd" which does not exist for me, so the dependency lib download fails....
X Description: Motivation and Context With a null PATH entry on Windows finding the mvn command resolves to absolute path: "C:\mvn.cmd" which does not exist for me, so the dependency lib downloa...
Opengraph URL: https://github.com/codellm-devkit/codeanalyzer-java/pull/127
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:f57222cb-a8cb-76bb-701f-d9b3bacec181 |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | E922:A9CDC:D42FEF:114014C:6990BD5B |
| html-safe-nonce | 9b5334cbaf11a44bbe18613e137b6dad28f06d8dfadba14435d06a9b4c7368cb |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFOTIyOkE5Q0RDOkQ0MkZFRjoxMTQwMTRDOjY5OTBCRDVCIiwidmlzaXRvcl9pZCI6Ijg2NjE3MzEzMjQzODAzNjQxMjMiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | cf0451bd3a74bf54a726e5353abb8875fb8ac026778ef039bdb736759549464e |
| hovercard-subject-tag | pull_request:2356985707 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/codellm-devkit/codeanalyzer-java/pull/127/checks |
| twitter:image | https://avatars.githubusercontent.com/u/4081634?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/4081634?s=400&v=4 |
| og:image:alt | Motivation and Context With a null PATH entry on Windows finding the mvn command resolves to absolute path: "C:\mvn.cmd" which does not exist for me, so the dependency lib download fails.... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| go-import | github.com/codellm-devkit/codeanalyzer-java git https://github.com/codellm-devkit/codeanalyzer-java.git |
| octolytics-dimension-user_id | 197800760 |
| octolytics-dimension-user_login | codellm-devkit |
| octolytics-dimension-repository_id | 731300153 |
| octolytics-dimension-repository_nwo | codellm-devkit/codeanalyzer-java |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 731300153 |
| octolytics-dimension-repository_network_root_nwo | codellm-devkit/codeanalyzer-java |
| turbo-body-classes | logged-out env-production page-responsive full-width full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width