Skip to content

fix: retain multiple command-line args for console script#1648

Merged
lvca merged 1 commit intoArcadeData:mainfrom
gramian:main
Jul 8, 2024
Merged

fix: retain multiple command-line args for console script#1648
lvca merged 1 commit intoArcadeData:mainfrom
gramian:main

Conversation

@gramian
Copy link
Collaborator

@gramian gramian commented Jul 8, 2024

What does this PR do?

This changes the way command-line arguments are passed from the console script to java (Linux and MacOS) from using $* (all arguments as one string) to $@ (arguments as array of strings), see https://tldp.org/LDP/abs/html/refcards.html .

Motivation

The change is necessary to be able to pass multiple arguments, otherwise the console cannot detect switches.

Additional Notes

I do not know and cannot test if this problem is also present on Windows.

Checklist

  • I have run the build using mvn clean package command
  • My unit tests cover both failure and success scenarios

@lvca lvca merged commit 60aa024 into ArcadeData:main Jul 8, 2024
@lvca
Copy link
Contributor

lvca commented Jul 8, 2024

Thanks!

@lvca lvca added the bug label Jul 8, 2024
@lvca lvca added this to the 24.6.1 milestone Jul 8, 2024
@lvca lvca added the fixed label Jul 23, 2024
mergify bot added a commit that referenced this pull request Jan 27, 2026
… in /studio in the build-tools group [skip ci]

Bumps the build-tools group in /studio with 1 update: [css-loader](https://github.com/webpack/css-loader).
Updates `css-loader` from 7.1.2 to 7.1.3
Release notes

*Sourced from [css-loader's releases](https://github.com/webpack/css-loader/releases).*

> v7.1.3
> ------
>
> ### [7.1.3](webpack/css-loader@v7.1.2...v7.1.3) (2026-01-27)
>
> ### Bug Fixes
>
> * allow to use `module` class name ([#1649](https://redirect.github.com/webpack/css-loader/issues/1649)) ([01869bc](webpack/css-loader@01869bc))
> * use official `createHash` for hashes ([#1618](https://redirect.github.com/webpack/css-loader/issues/1618)) ([06587e5](webpack/css-loader@06587e5))
> * use official `hash*` options for hashes ([#1619](https://redirect.github.com/webpack/css-loader/issues/1619)) ([9544c3e](webpack/css-loader@9544c3e))


Changelog

*Sourced from [css-loader's changelog](https://github.com/webpack/css-loader/blob/main/CHANGELOG.md).*

> ### [7.1.3](webpack/css-loader@v7.1.2...v7.1.3) (2026-01-27)
>
> ### Bug Fixes
>
> * allow to use `module` class name ([#1649](https://redirect.github.com/webpack/css-loader/issues/1649)) ([01869bc](webpack/css-loader@01869bc))
> * use official `createHash` for hashes ([#1618](https://redirect.github.com/webpack/css-loader/issues/1618)) ([06587e5](webpack/css-loader@06587e5))
> * use official `hash*` options for hashes ([#1619](https://redirect.github.com/webpack/css-loader/issues/1619)) ([9544c3e](webpack/css-loader@9544c3e))


Commits

* [`b2b2de7`](webpack/css-loader@b2b2de7) chore(release): 7.1.3
* [`01869bc`](webpack/css-loader@01869bc) fix: allow to use `module` class name ([#1649](https://redirect.github.com/webpack/css-loader/issues/1649))
* [`7dd15ec`](webpack/css-loader@7dd15ec) chore(deps): bump js-yaml ([#1648](https://redirect.github.com/webpack/css-loader/issues/1648))
* [`db26202`](webpack/css-loader@db26202) chore(deps-dev): bump lodash from 4.17.21 to 4.17.23 ([#1647](https://redirect.github.com/webpack/css-loader/issues/1647))
* [`7daf1b8`](webpack/css-loader@7daf1b8) Update CONTRIBUTING link to point to GitHub page
* [`de1e633`](webpack/css-loader@de1e633) chore: correct link path ([#1645](https://redirect.github.com/webpack/css-loader/issues/1645))
* [`563ad63`](webpack/css-loader@563ad63) chore: migrate from contrib and swap branches ([#1644](https://redirect.github.com/webpack/css-loader/issues/1644))
* [`e68bf7e`](webpack/css-loader@e68bf7e) chore: update github actions/checkout from v4 to v5 ([#1642](https://redirect.github.com/webpack/css-loader/issues/1642))
* [`99eb892`](webpack/css-loader@99eb892) chore: update jest from v29 to v30 ([#1637](https://redirect.github.com/webpack/css-loader/issues/1637))
* [`af6652f`](webpack/css-loader@af6652f) docs: improving typos in ISSUE\_TEMPLATE files ([#1636](https://redirect.github.com/webpack/css-loader/issues/1636))
* Additional commits viewable in [compare view](webpack/css-loader@v7.1.2...v7.1.3)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=css-loader&package-manager=npm\_and\_yarn&previous-version=7.1.2&new-version=7.1.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  major version` will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
- `@dependabot ignore  minor version` will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
- `@dependabot ignore ` will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
- `@dependabot unignore ` will remove all of the ignore conditions of the specified dependency
- `@dependabot unignore  ` will remove the ignore condition of the specified dependency and ignore conditions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments