Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IDEasy not cloning active repositories from devonfw-ide legacy settings #911

Open
hohwille opened this issue Jan 9, 2025 · 0 comments
Open
Labels
bug Something isn't working configuration should be configurable or configuration change repository Commandlet to clone, build or import git repositories settings ide-settings repo and replated processes and features

Comments

@hohwille
Copy link
Member

hohwille commented Jan 9, 2025

Expected behavior

As a IDEasy user, I want to work with my devonfw-ide project or upgrade-settings to IDEasy so that it works like with devonfw-ide and I do not have to worry about incompatibilities.

Actual behavior

Invalid repository configuration xyz.properties - both 'path' and 'git-url' have to be defined.

Steps to reproduce (bug) / Use Case of feature request (enhancement)

  1. ide create myproject «url-of-devonfw-ide-settings-project-with-active-project-repository»

Related/Dependent Issues

Story #100, PR #143

Implemented like this:

properties.getProperty("workspace"), properties.getProperty("git_url"), properties.getProperty("git_branch"),

Correctly as documented in devonfw-ide and doc was linked from the story.

Bute there is something I missed:
https://github.com/devonfw/ide/blob/8698d2ba7468d34aec82c86fabdb09b5ee84d9c8/scripts/src/main/resources/scripts/functions-core#L41

When devonfw-ide started, a dot was used as separator (git.url).
Later properties were loaded into bash variables and those variables cannot have a dot in the name.
So we replaced the dot with an underscore.
All projects that started earlier were using git.url and git.branch instead of git_url and git_branch.
See here as a proof:
https://github.com/devonfw/ide-settings/blob/4214b9dca2b3d5efdcbf8e60cb62617cc90f2f9f/projects/devon4j.properties

Comments/Hints:

  1. We need to add compatibility support in RepositoryConfig to fallback to the property names with dots.
  2. We IMHO should improve #759: upgrade settings commandlet #820 to migrate these properties from dot to underscores, since this is the officially specified format according to our documentation.

Affected version:

  • 2024.12.002-beta
@hohwille hohwille added bug Something isn't working configuration should be configurable or configuration change repository Commandlet to clone, build or import git repositories settings ide-settings repo and replated processes and features labels Jan 9, 2025
@github-project-automation github-project-automation bot moved this to 🆕 New in IDEasy board Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working configuration should be configurable or configuration change repository Commandlet to clone, build or import git repositories settings ide-settings repo and replated processes and features
Projects
Status: 🆕 New
Development

No branches or pull requests

1 participant