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

add git-tfs 0.34.0 #153871

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add git-tfs 0.34.0 #153871

wants to merge 1 commit into from

Conversation

pmiossec
Copy link

@pmiossec pmiossec commented May 16, 2024

Checklist for Pull Requests

Manifests

  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • This PR only modifies one (1) manifest
  • Have you validated your manifest locally with winget validate --manifest <path>?
  • Have you tested your manifest locally with winget install --manifest <path>?
  • Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.


Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wingetbot wingetbot added New-Package Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Completed Validation passed labels May 16, 2024
@stephengillie
Copy link
Collaborator

stephengillie commented May 16, 2024

Hi @pmiossec,

This package installed normally, but had a few issues running:
image
git-tfs_log.txt

  • First is a known issue where the package manager creates a symlink to the application, and while the application runs, the symlink does not. This is usually due to the "start in" folder being the symlink location instead of the application location, preventing the application from reaching files it needs.
  • Second, this application gave an error when launched from its folder. Is this expected? Edit: Should Microsoft.VisualStudioCode or similar be a dependency?

@stephengillie stephengillie added the zip-binary Depends on library files that aren't available via symlink label May 16, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added Blocking-Issue Manifest validation is blocked by a known issue. Needs-Author-Feedback This needs a response from the author. labels May 16, 2024
@pmiossec
Copy link
Author

@stephengillie Thanks for the feedback

* First is a known issue where the package manager creates a symlink to the application, and while the application runs, the symlink does not. This is usually due to the "start in" folder being the symlink location instead of the application location, preventing the application from reaching files it needs.

What do you recommend to solve the issue?
Removing the symlink is enough? Or there is a better solution?
I have not found a clear solution...

* Second, this application gave an error when launched from its folder. Is this expected? Edit: Should `Microsoft.VisualStudioCode` or similar be a dependency?

There is a dependency on "Microsoft Visual Studio" (not Code 😉 ).
But we can't enforce a dependency for a specific version because normally, the developer already has a version installed and it could be any version in 2015, 2017, 2019 or 2022. git-tfs successfully load the dependency dll depending on the version installed.
It could be handled a little more gracefully but I don't think we should block the merge for this reason (the behavior is like that for years and we never had a complain because it's a known dependency by the user)

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels May 17, 2024
@stephengillie
Copy link
Collaborator

A bot is supposed to link this issue when the zip-binary Depends on library files that aren't available via symlink label is applied:

@pmiossec
Copy link
Author

A bot is supposed to link this issue when the zip-binary label is applied:

It did 😉
That's why I said that "I have not found a clear solution" 😟
A lot of messages and talking about different subject (auto-unpack exe, bat file) bat without a message providing a clear answer of a solution to follow...

As I said, I think I will remove the symlink because I don't see an added value. Is it ok?

@stephengillie
Copy link
Collaborator

stephengillie commented May 17, 2024

As I said, I think I will remove the symlink because I don't see an added value. Is it ok?

If it's possible to remove, I would be very interested in learning how.

@pmiossec
Copy link
Author

As I said, I think I will remove the symlink because I don't see an added value. Is it ok?

If it's possible to remove, I would be very interested in learning how.

Indeed. A previous attempt made me think that not adding PortableCommandAlias: line will not create the symlink but a quick try confirm your thought, the symlink is created. A dirty (working) workaround is to name it something like PortableCommandAlias: _git-tfs_DONT_USE_THIS_ALIAS...

@stephengillie stephengillie removed Validation-Completed Validation passed Needs-Attention This work item needs to be reviewed by a member of the core team. labels May 20, 2024
@pmiossec
Copy link
Author

pmiossec commented Jun 1, 2024

@stephengillie Do you think it's an acceptable (dirty) workaround?
If yes, I push it. Otherwise, I don't know how to go forward...

@stephengillie
Copy link
Collaborator

Hi @pmiossec,

Sorry to miss this. Without the symlink alias, how will users find the application? Some of our users might be people who install random applications for a hobby, and some might install this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Blocking-Issue Manifest validation is blocked by a known issue. New-Package zip-binary Depends on library files that aren't available via symlink
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants