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

(datagrip) When installing a new Datagrip minor version (e.g. 2023.2.0), an empty directory under Program Files (x86)\JetBrains\DataGrip 2023.2.0 is created #2257

Closed
9numbernine9 opened this issue Jul 21, 2023 · 9 comments · Fixed by #2258 · May be fixed by #2366

Comments

@9numbernine9
Copy link

Chocolatey Version

2.1.0

Chocolatey License

None

Package Version

2023.2.0

Current Behaviour

Hello! 👋

When installing the latest version of DataGrip (2023.2.0), line 9 sets the $installDir variable to $programFiles\JetBrains\DataGrip $env:ChocolateyPackageVersion. Unfortunately, this value is not quite correct for new DataGrip minor versions: when a DataGrip version number ends in .0, the directory that the DataGrip installer will install itself into will omit the .0 patch version.

The end result of this is that we end up with two directories under Program Files (x86):

  • Program Files (x86)\JetBrains\DataGrip 2023.2, where the installer installed itself and placed all the code.
  • Program Files (x86)\JetBrains\DataGrip 2023.2.0, an empty directory created by the ChocolateyInstall.ps1 script.

Although this doesn't cause any errors, it is slightly confusing.

Expected Behaviour

Only one directory for a DataGrip install should be created, not two.

Steps To Reproduce

  • Install DataGrip 2023.2.0 via Chocolatey.
  • In Windows Explorer, go to C:\Program Files (x86)\JetBrains
  • Observe that there are two similar directories: DataGrip 2023.2 ✅ and DataGrip 2023.2.0

Environment

- Operating System: Microsoft Windows NT 10.0.19045.0
- PowerShell Version: 5.1.19041.3031
- Shell: cmd

Chocolatey Log

N/A

Anything else?

No response

pauby added a commit to pauby/chocolatey-coreteampackages that referenced this issue Jul 22, 2023
The workaround, and folder having to be created before the installer is run, can be removed as the issue has been fixed.
@pauby
Copy link
Member

pauby commented Jul 22, 2023

I've raised #2258 to fix this.

@9numbernine9 9numbernine9 changed the title (datagrip) When installing a new Datagrip minor version (e.g. 2023.2.0), an empty directory under Program Files (x86)\JetBrains\DataGrip 2023.2.0 is crreated (datagrip) When installing a new Datagrip minor version (e.g. 2023.2.0), an empty directory under Program Files (x86)\JetBrains\DataGrip 2023.2.0 is created Jul 22, 2023
@9numbernine9
Copy link
Author

@pauby Is it too late to expand the scope of this issue? I just noticed that the exact same thing is happening with the WebStorm 2023.2.0 package - it stands to reason the packages for the other JetBrains products are likely affected as well.

@pauby
Copy link
Member

pauby commented Jul 31, 2023

@9numbernine9 We prefer one PR per package but I think in this case it would make sense to have the same change, for multiple packages, in one PR.

If you want to amend the title, and the original comment on this issue, and add checkboxes for the problematic packages, I can look at amending the PR. In the meantime I'll put #2258 in draft.

@9numbernine9
Copy link
Author

9numbernine9 commented Jul 31, 2023

@pauby I've looked through all of the Jetbrains packages (I think?) in the Chocolatey community repository and the only two that I think actually have this issue are datagrip and webstorm. With that in mind, I'll just submit a similar issue for webstorm and then we can uphold the "one package per issue" ideal. 😄

@github-actions
Copy link

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
This issue will be closed in 14 days if it continues to be inactive.

@9numbernine9
Copy link
Author

Still relevant.

Copy link

github-actions bot commented Dec 4, 2023

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
This issue will be closed in 14 days if it continues to be inactive.

Please do not add a comment to circumvent automatic closure unless you plan to help move it forward.
Doing this may lead to the issue being closed immediately instead.

9numbernine9 added a commit to 9numbernine9/choco-packages that referenced this issue Dec 7, 2023
When determining the installation directory for Datagrip, remove
any `.0` suffix that might be present in the version number.
Jetbrains doesn't use this part of the release version number when
installing, so this causes Chocolatey to create an empty
directory that never gets used or cleaned up.

Fixes issue chocolatey-community#2257 .
Copy link

Dear contributor,

As this issue seems to have been inactive for quite some time now, I've automatically closed it.
If you feel this is a valid issue, please feel free to re-open the issue if/when a pull request
has been added.
Thank you for your contribution.

@9numbernine9
Copy link
Author

Still relevant - PR was raised so I'm not sure why this issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants