Skip to content

chore: Rework starting scripts for Windows#738

Merged
lvca merged 2 commits into
ArcadeData:mainfrom
littleli:chore/windows-scripts
Jan 4, 2023
Merged

chore: Rework starting scripts for Windows#738
lvca merged 2 commits into
ArcadeData:mainfrom
littleli:chore/windows-scripts

Conversation

@littleli
Copy link
Copy Markdown
Contributor

@littleli littleli commented Jan 3, 2023

What does this PR do?

This PR improves following:

  • covers server.bat, console.bat and even stresstester.bat
  • fixes syntactic issues in batch files for Windows when lookup up for java executable
  • now it searches for java executable on PATH
  • prints actual resolved ARCADEDB_HOME and script path to output at launch
  • always changes directory to ARCADEDB_HOME so database files, logs and config directories are used correctly even if console.bat or server.bat are launched from a different current directory
  • Inspiration was taken partially from Maven's mvn.cmd but only a little bit

Motivation

Improve experience for Windows developers.
I'm able to create a manifest for Scoop installer and make ArcadeDB installable using simple scoop install arcadedb once we have launch scripts in a good shape.

Additional Notes

My appologies I actually touched license blocks. There was one duplicate in console.bat and conflicting copyright on stresstester.bat. I expect request for change with suggestion during review.

Also I was unable to test stresstester.bat as I'm missing some classes.

I changed arcadedb.com to https://arcadedb.com intentionally on console output reason being terminals actually can make this link clickable.

Demo

Launch server

arcadedb-server

Launch console

arcadedb-console

Checklist

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

@lvca lvca added bug labels Jan 4, 2023
@lvca lvca added this to the 23.1.1 milestone Jan 4, 2023
@lvca lvca merged commit 7f28711 into ArcadeData:main Jan 4, 2023
@lvca
Copy link
Copy Markdown
Member

lvca commented Jan 4, 2023

Congrats Ales on your first PR!

Is there any chance you can also fix the banner? For some reason doesn't work with .bat. This is on server.sh:

echo ""
echo " █████╗ ██████╗  ██████╗ █████╗ ██████╗ ███████╗██████╗ ██████╗"
echo "██╔══██╗██╔══██╗██╔════╝██╔══██╗██╔══██╗██╔════╝██╔══██╗██╔══██╗"
echo "███████║██████╔╝██║     ███████║██║  ██║█████╗  ██║  ██║██████╔╝"
echo "██╔══██║██╔══██╗██║     ██╔══██║██║  ██║██╔══╝  ██║  ██║██╔══██╗"
echo "██║  ██║██║  ██║╚██████╗██║  ██║██████╔╝███████╗██████╔╝██████╔╝"
echo "╚═╝  ╚═╝╚═╝  ╚═╝ ╚═════╝╚═╝  ╚═╝╚═════╝ ╚══════╝╚═════╝ ╚═════╝"
echo "PLAY WITH DATA                                    arcadedb.com"

mergify Bot added a commit that referenced this pull request Sep 14, 2025
…6.0 to 3.6.1 [skip ci]

Bumps [org.apache.maven.plugins:maven-shade-plugin](https://github.com/apache/maven-shade-plugin) from 3.6.0 to 3.6.1.
Release notes

*Sourced from [org.apache.maven.plugins:maven-shade-plugin's releases](https://github.com/apache/maven-shade-plugin/releases).*

> 3.6.1
> -----
>
> 📝 Documentation updates
> -----------------------
>
> * [[MNGSITE-529]](https://issues.apache.org/jira/browse/MNGSITE-529) - Rename "Goals" to "Plugin Documentation" ([#250](https://redirect.github.com/apache/maven-shade-plugin/pull/250)) [`@​Bukama`](https://github.com/Bukama)
>
> 👻 Maintenance
> -------------
>
> * Enable prevent branch protection rules ([#746](https://redirect.github.com/apache/maven-shade-plugin/pull/746)) [`@​sparsick`](https://github.com/sparsick)
> * Enable GH issues ([#253](https://redirect.github.com/apache/maven-shade-plugin/pull/253)) [`@​Bukama`](https://github.com/Bukama)
> * Add missing [`@​Override`](https://github.com/Override) annotations ([#246](https://redirect.github.com/apache/maven-shade-plugin/pull/246)) [`@​elharo`](https://github.com/elharo)
> * Merge `ApacheLicenseResourceTransformer` tests ([#245](https://redirect.github.com/apache/maven-shade-plugin/pull/245)) [`@​Goooler`](https://github.com/Goooler)
> * Add test cases for .md supports in the Apache License and Notice transformers ([#243](https://redirect.github.com/apache/maven-shade-plugin/pull/243)) [`@​Goooler`](https://github.com/Goooler)
> * [[MSHADE-479]](https://issues.apache.org/jira/browse/MSHADE-479) - Make the mojo much less noisy ([#233](https://redirect.github.com/apache/maven-shade-plugin/pull/233)) [`@​elharo`](https://github.com/elharo)
>
> 📦 Dependency updates
> --------------------
>
> * Bump org.codehaus.mojo:mrm-maven-plugin from 1.6.0 to 1.7.0 ([#748](https://redirect.github.com/apache/maven-shade-plugin/pull/748)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.hamcrest:hamcrest-core from 2.2 to 3.0 ([#235](https://redirect.github.com/apache/maven-shade-plugin/pull/235)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.apache.maven.plugins:maven-plugins from 42 to 45 ([#738](https://redirect.github.com/apache/maven-shade-plugin/pull/738)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.apache.commons:commons-compress from 1.26.2 to 1.28.0 ([#743](https://redirect.github.com/apache/maven-shade-plugin/pull/743)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.xmlunit:xmlunit-legacy from 2.10.0 to 2.10.3 ([#745](https://redirect.github.com/apache/maven-shade-plugin/pull/745)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump ASM 9.8 to support JDK 25 bytecode ([#744](https://redirect.github.com/apache/maven-shade-plugin/pull/744)) [`@​pan3793`](https://github.com/pan3793)
> * Bump commons-io:commons-io from 2.13.0 to 2.14.0 in /src/it/projects/MSHADE-105/shaded-jar ([#241](https://redirect.github.com/apache/maven-shade-plugin/pull/241)) @[dependabot[bot]](https://github.com/apps/dependabot)


Commits

* [`9662c98`](apache/maven-shade-plugin@9662c98) [maven-release-plugin] prepare for next development iteration
* [`06902bd`](apache/maven-shade-plugin@06902bd) [maven-release-plugin] prepare release maven-shade-plugin-3.6.1
* [`29e9a9d`](apache/maven-shade-plugin@29e9a9d) add .git
* [`844e61a`](apache/maven-shade-plugin@844e61a) use github
* [`43101f9`](apache/maven-shade-plugin@43101f9) [maven-release-plugin] prepare release maven-shade-plugin-3.6.1
* [`2ffb28d`](apache/maven-shade-plugin@2ffb28d) use release drafter v4 and dependabot to updade gha as well ([#750](https://redirect.github.com/apache/maven-shade-plugin/issues/750))
* [`f5b590e`](apache/maven-shade-plugin@f5b590e) Bump org.codehaus.mojo:mrm-maven-plugin from 1.6.0 to 1.7.0
* [`eee0319`](apache/maven-shade-plugin@eee0319) Bump org.hamcrest:hamcrest-core from 2.2 to 3.0
* [`0d5a7a4`](apache/maven-shade-plugin@0d5a7a4) fix upgrade, remove those useless final
* [`6e5f0f3`](apache/maven-shade-plugin@6e5f0f3) Bump org.apache.maven.plugins:maven-plugins from 42 to 45
* Additional commits viewable in [compare view](apache/maven-shade-plugin@maven-shade-plugin-3.6.0...v3.6.1)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=org.apache.maven.plugins:maven-shade-plugin&package-manager=maven&previous-version=3.6.0&new-version=3.6.1)](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)
mergify Bot added a commit that referenced this pull request Mar 22, 2026
Bumps [dorny/test-reporter](https://github.com/dorny/test-reporter) from 2.6.0 to 3.0.0.
Release notes

*Sourced from [dorny/test-reporter's releases](https://github.com/dorny/test-reporter/releases).*

> v3.0.0
> ------
>
> **Note:** The v3 release requires NodeJS 24 runtime on GitHub Actions runners.
>
> What's Changed
> --------------
>
> * Upgrade action runtime to Node.js 24 by [`@​dav-tb`](https://github.com/dav-tb) in [dorny/test-reporter#738](https://redirect.github.com/dorny/test-reporter/pull/738)
> * Explicitly use lowest permissions required to run workflow by [`@​jozefizso`](https://github.com/jozefizso) in [dorny/test-reporter#745](https://redirect.github.com/dorny/test-reporter/pull/745)
>
> ### Other Changes
>
> * Bump `@​typescript-eslint/parser` from 8.57.0 to 8.57.1 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#742](https://redirect.github.com/dorny/test-reporter/pull/742)
> * Bump `@​types/adm-zip` from 0.5.7 to 0.5.8 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#743](https://redirect.github.com/dorny/test-reporter/pull/743)
> * Bump flatted from 3.4.1 to 3.4.2 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#744](https://redirect.github.com/dorny/test-reporter/pull/744)
>
> New Contributors
> ----------------
>
> * [`@​dav-tb`](https://github.com/dav-tb) made their first contribution in [dorny/test-reporter#738](https://redirect.github.com/dorny/test-reporter/pull/738)
>
> **Full Changelog**: <dorny/test-reporter@v2.7.0...v3.0.0>
>
> v2.7.0
> ------
>
> What's Changed
> --------------
>
> * Feature: Add `slug-prefix` output for link anchors [dorny/test-reporter#731](https://redirect.github.com/dorny/test-reporter/pull/731)
> * Feature: Report `jest-junit` testsuite errors as failures [dorny/test-reporter#155](https://redirect.github.com/dorny/test-reporter/pull/155)
> * Security: Update dependencies to fix reported security vulnerabilities
>
> ### Other Changes
>
> * Bump eslint from 9.39.3 to 9.39.4 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#732](https://redirect.github.com/dorny/test-reporter/pull/732)
> * Bump `@​typescript-eslint/parser` from 8.56.1 to 8.57.0 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#733](https://redirect.github.com/dorny/test-reporter/pull/733)
> * Bump jest from 30.2.0 to 30.3.0 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#734](https://redirect.github.com/dorny/test-reporter/pull/734)
> * Bump undici from 6.23.0 to 6.24.1 by [`@​dependabot`](https://github.com/dependabot)[bot] in [dorny/test-reporter#736](https://redirect.github.com/dorny/test-reporter/pull/736)
> * Update `flatted` package to v3.4.1 to fix a vulnerability by [`@​jozefizso`](https://github.com/jozefizso) in [dorny/test-reporter#739](https://redirect.github.com/dorny/test-reporter/pull/739)
>
> New Contributors
> ----------------
>
> * [`@​dt-thomas-durand`](https://github.com/dt-thomas-durand) made their first contribution in [dorny/test-reporter#731](https://redirect.github.com/dorny/test-reporter/pull/731)
> * [`@​johnbartholomew`](https://github.com/johnbartholomew) made their first contribution in [dorny/test-reporter#155](https://redirect.github.com/dorny/test-reporter/pull/155)
>
> **Full Changelog**: <dorny/test-reporter@v2.6.0...v2.7.0>


Changelog

*Sourced from [dorny/test-reporter's changelog](https://github.com/dorny/test-reporter/blob/main/CHANGELOG.md).*

> Changelog
> =========
>
> 3.0.0
> -----
>
> * Feature: Use NodeJS 24 LTS as default runtime [dorny/test-reporter#738](https://redirect.github.com/dorny/test-reporter/pull/738)
>
> 2.7.0
> -----
>
> * Feature: Add `slug-prefix` output for link anchors [dorny/test-reporter#731](https://redirect.github.com/dorny/test-reporter/pull/731)
> * Feature: Report `jest-junit` testsuite errors as failures [dorny/test-reporter#155](https://redirect.github.com/dorny/test-reporter/pull/155)
> * Security: Update dependencies to fix reported security vulnerabilities
>
> 2.6.0
> -----
>
> * Fix: For `workflow_run` events, resolve the commit of the check run from related pull request head commits first (matching `workflow_run.head_branch`, then first PR), and fall back to `workflow_run.head_sha` for non-PR runs [dorny/test-reporter#673](https://redirect.github.com/dorny/test-reporter/pull/673)
> * Change: The `test-reporter` action will listed all artifacts associated with the build run [dorny/test-reporter#693](https://redirect.github.com/dorny/test-reporter/pull/693)
> * Maintenance: Upgrade to ESLint v9 [dorny/test-reporter#629](https://redirect.github.com/dorny/test-reporter/pull/629)
>
> 2.5.0
> -----
>
> * Feature: Add Nette Tester support with `tester-junit` reporter [dorny/test-reporter#707](https://redirect.github.com/dorny/test-reporter/pull/707)
> * Maintenance: Bump actions/upload-artifact from 5 to 6 [dorny/test-reporter#695](https://redirect.github.com/dorny/test-reporter/pull/695)
>
> 2.4.0
> -----
>
> * Feature: Add PHPUnit support with JUnit XML dialect parser [dorny/test-reporter#422](https://redirect.github.com/dorny/test-reporter/pull/422)
> * Feature: Add JUnit XML sample files and tests for validation [dorny/test-reporter#701](https://redirect.github.com/dorny/test-reporter/pull/701)
> * Fix: Refactor deprecated `String.substr()` function to use `String.substring()` [dorny/test-reporter#704](https://redirect.github.com/dorny/test-reporter/pull/704)
>
> 2.3.0
> -----
>
> * Feature: Add Python support with `python-xunit` reporter (pytest) [dorny/test-reporter#643](https://redirect.github.com/dorny/test-reporter/pull/643)
> * Feature: Add pytest traceback parsing and `directory-mapping` option [dorny/test-reporter#238](https://redirect.github.com/dorny/test-reporter/pull/238)
> * Performance: Update sax.js to fix large XML file parsing [dorny/test-reporter#681](https://redirect.github.com/dorny/test-reporter/pull/681)
> * Documentation: Complete documentation for all supported reporters [dorny/test-reporter#691](https://redirect.github.com/dorny/test-reporter/pull/691)
> * Security: Bump js-yaml and mocha in /reports/mocha (fixes prototype pollution) [dorny/test-reporter#682](https://redirect.github.com/dorny/test-reporter/pull/682)
>
> 2.2.0
> -----
>
> * Feature: Add collapsed option to control report summary visibility [dorny/test-reporter#664](https://redirect.github.com/dorny/test-reporter/pull/664)
> * Fix badge encoding for values including underscore and hyphens [dorny/test-reporter#672](https://redirect.github.com/dorny/test-reporter/pull/672)
> * Fix missing `report-title` attribute in action definition [dorny/test-reporter#637](https://redirect.github.com/dorny/test-reporter/pull/637)
> * Refactor variable names to fix shadowing issues [dorny/test-reporter#630](https://redirect.github.com/dorny/test-reporter/pull/630)
>
> 2.1.1
> -----
>
> * Fix error when a TestMethod element does not have a className attribute in a trx file [dorny/test-reporter#623](https://redirect.github.com/dorny/test-reporter/pull/623)
> * Add stack trace from trx to summary [dorny/test-reporter#615](https://redirect.github.com/dorny/test-reporter/pull/615)
> * List only failed tests [dorny/test-reporter#606](https://redirect.github.com/dorny/test-reporter/pull/606)
> * Add type definitions to `github-utils.ts` [dorny/test-reporter#604](https://redirect.github.com/dorny/test-reporter/pull/604)
> * Avoid split on undefined [dorny/test-reporter#258](https://redirect.github.com/dorny/test-reporter/pull/258)
> * Return links to summary report [dorny/test-reporter#588](https://redirect.github.com/dorny/test-reporter/pull/588)
> * Add step summary short summary [dorny/test-reporter#589](https://redirect.github.com/dorny/test-reporter/pull/589)
> * Fix for empty TRX TestDefinitions [dorny/test-reporter#582](https://redirect.github.com/dorny/test-reporter/pull/582)
> * Increase step summary limit to 1MiB [dorny/test-reporter#581](https://redirect.github.com/dorny/test-reporter/pull/581)
> * Fix input description for list options [dorny/test-reporter#572](https://redirect.github.com/dorny/test-reporter/pull/572)
>
> 2.1.0
> -----

... (truncated)


Commits

* [`a43b3a5`](dorny/test-reporter@a43b3a5) test-reporter release v3.0.0
* [`79216dc`](dorny/test-reporter@79216dc) Merge pull request [#744](https://redirect.github.com/dorny/test-reporter/issues/744) from dorny/dependabot/npm\_and\_yarn/flatted-3.4.2
* [`62c3873`](dorny/test-reporter@62c3873) Merge pull request [#745](https://redirect.github.com/dorny/test-reporter/issues/745) from dorny/feature/workflow\_permissions
* [`bd74ca6`](dorny/test-reporter@bd74ca6) Explicitly use lowest permissions required to run workflow
* [`0c2b2aa`](dorny/test-reporter@0c2b2aa) Merge pull request [#738](https://redirect.github.com/dorny/test-reporter/issues/738) from dav-tb/feature/node24-runtime
* [`f7eee30`](dorny/test-reporter@f7eee30) Use the `@types/node` definitions for NodeJS 24 runtime
* [`ad0a96f`](dorny/test-reporter@ad0a96f) Upgrade action runtime to Node.js 24
* [`543415c`](dorny/test-reporter@543415c) Merge pull request [#741](https://redirect.github.com/dorny/test-reporter/issues/741) from dorny/release/v2.7.0
* [`f1284d0`](dorny/test-reporter@f1284d0) Bump flatted from 3.4.1 to 3.4.2
* [`3c1b53f`](dorny/test-reporter@3c1b53f) Merge pull request [#743](https://redirect.github.com/dorny/test-reporter/issues/743) from dorny/dependabot/npm\_and\_yarn/types/adm-zip-0.5.8
* Additional commits viewable in [compare view](dorny/test-reporter@3d76b34...a43b3a5)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=dorny/test-reporter&package-manager=github\_actions&previous-version=2.6.0&new-version=3.0.0)](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 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)
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