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

Errors are printed to standard output #483

Closed
jakub-bochenski opened this issue Sep 26, 2017 · 8 comments · Fixed by #490
Closed

Errors are printed to standard output #483

jakub-bochenski opened this issue Sep 26, 2017 · 8 comments · Fixed by #490
Labels

Comments

@jakub-bochenski
Copy link

Running the powershell appimage on centos (no x-window installed) results in the following extra messages:

    ./PowerShell-6.0.0-beta.7-x86_64.AppImage
    which: no desktop-file-validate in (/tmp/.mount_ZfbAnL/usr/bin/:/home/acme:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/home/acme/.local/bin:/home/acme/bin)
    desktop-file-validate is missing. Skipping /tmp/.mount_ZfbAnL/usr/bin/powershell.wrapper.
    PowerShell v6.0.0-beta.7
    Copyright (C) Microsoft Corporation. All rights reserved.
    The specified drive root "net:[4026532497]" either does not exist, or it is not a folder.

Running it with stderr redirected to /dev/null still leaves desktop-file-validate is missing. Skipping /tmp/.mount_R39YOm/usr/bin/powershell.wrapper. in the output.

    [acme@iq-jenkins-dockerbuilder-2 ~]$ ./PowerShell-6.0.0-beta.7-x86_64.AppImage 2>/dev/null
    desktop-file-validate is missing. Skipping /tmp/.mount_R39YOm/usr/bin/powershell.wrapper.
    PowerShell v6.0.0-beta.7
    Copyright (C) Microsoft Corporation. All rights reserved.

It should go to stderr instead.

@probonopd
Copy link
Member

probonopd commented Sep 26, 2017

Please report this to the PowerShell project, thanks.

@probonopd probonopd reopened this Sep 26, 2017
@probonopd
Copy link
Member

Seems to be a bug in the desktopintegration bash script. Needs to be fixed here, and then needs to be updated in the PowerShell project.

@TheAssassin
Copy link
Member

@probonopd we should rather attempt to merge #433.

@probonopd probonopd added the bug label Sep 26, 2017
@jakub-bochenski
Copy link
Author

@probonopd @TheAssassin can you give me some idea when this will be fixed?
It's rather annoying to have to filter out the output on each invocation of powershell. Moving output from stdout to stderr seems like an easy fix.

TheAssassin added a commit that referenced this issue Oct 11, 2017
@TheAssassin
Copy link
Member

@jakub-bochenski I opened a pull request (#490). We will have to test whether the version in there works with the PowerShell AppImage.

However, the build in there is failing, for some totally unrelated reason. @probonopd will have to merge it, as I can't fix this issue.

@jakub-bochenski
Copy link
Author

@TheAssassin I see the PR is merged to develop.
Do you think I should try the continuous build of AppImage with Powershell?

@probonopd
Copy link
Member

The PowerShell team would need to carry the changes over, it's just a couple of changes in a bash script. The PowerShell team is not automatically fetching the latest master from this project.

@TheAssassin
Copy link
Member

I agree. I fixed the upstream issue, but we can't trigger updates for other projects who would potentially benefit from the fix. Please step up to them and ask them to update the tools they use.

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

Successfully merging a pull request may close this issue.

3 participants