GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Remove the workaround to enable macOS launcher · Issue #5262 · PowerShell/PowerShell

Remove the workaround to enable macOS launcher #5262

New issue
Copy link
New issue
Copy link
Closed
Closed
Remove the workaround to enable macOS launcher#5262
Copy link
Labels
Resolution-FixedThe issue is fixed.The issue is fixed.

Description

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan)
opened on Oct 27, 2017
Issue body actions

The change was reverted to unblock PRs being blocked in packaging.

#5259

Further investigation should be done to re-enable macOS launcher.

Activity

  1. Aditya Patwardhan (adityapatwardhan)
    changed the title [-]Remove the workaround to enbable macOS launcher[/-] [+]Remove the workaround to enable macOS launcher[/+] on Oct 27, 2017
  2. thezim commented on Oct 29, 2017

    @thezim
    Greg Zimmerman (thezim)
    on Oct 29, 2017
    Contributor
    More actions

    Looks like a permissions issue to me. Is the account doing the build (owner) the same as who created the inital file structure. I could rewrite it so it builds the app in a user space temp folder such as TMPDIR to avoid conflicts.

  3. adityapatwardhan commented on Oct 30, 2017

    @adityapatwardhan
    Aditya Patwardhan (adityapatwardhan)
    on Oct 30, 2017
    MemberAuthor
    More actions

    Greg Zimmerman (@thezim) yes it is the same account. It would be great if you could send a PR to resolve this.

  4. thezim commented on Oct 31, 2017

    @thezim
    Greg Zimmerman (thezim)
    on Oct 31, 2017
    Contributor
    More actions

    Issue

    Aditya Patwardhan (@adityapatwardhan) so I found out what is going on. When the the installation of PowerShell occurs it is actually recognizing the app that exists in the packaging folder and updating the copy there tools/packaging/macos/launcher/ROOT/Applications/PowerShell.app instead of installing it to /Applications as you would think. Since it gets installed as root:wheel 755 you lose access to the files.

    The behavior may seem like madness but it makes sense as apps can be installed pretty much anywhere you like. Being able automatically find these instances is beneficial.

    This issue only occurs on a machine with build on it. I verified that a vanilla High Sierra does not have this issue with just the installation from .pkg.

    I'm afraid moving it to different temporary folder will probably not work as it will get picked up there too. My intuition is one the the identifiers in Info.plist is triggering its pickup. Removing the tools/packaging/macos/launcher/ROOT/Applications/PowerShell.app folder prior to installation does fix the issue.

    Possible solutions

    1. Remove the macOS app folder just after fpm runs here. Not the best but maybe good enough for getting past CI.
    2. Have packaging build the macOS app procedurally as it is simple a few files and folders then remove the folder after packaging, same as in 1. This has the advantage of not having the folder structure and files in the repo directly as they can be generated at will.
    3. Figure out what is triggering the capture of the build staging launcher and make a change to the plist and/or structure to prevent if from being picked up post fpm.

    Also for anyone wanting to try to repro this to uninstall PowerShell from macOS you should be using the following. I will get documentation updated soon with the below.

    sudo rm -rf /usr/local/microsoft /usr/local/bin/pwsh /Applications/PowerShell.app /usr/local/share/man/man1/pwsh.1.gz
    sudo pkgutil --forget powershell

    /cc Travis Plunk (@TravisEz13)

  5. TravisEz13 commented on Oct 31, 2017

    @TravisEz13
    Travis Plunk (TravisEz13)
    on Oct 31, 2017
    Member
    More actions

    Greg Zimmerman (@thezim) If you enable travis-ci or your branch, it will attempt to build the package each time you attempt to push. I don't have any preference between 1 & 2. Although, we are getting close to release. We should probably be looking for a simple solution. It seems to me 1 is simpler (if it works.)

  6. thezim commented on Nov 1, 2017

    @thezim
    Greg Zimmerman (thezim)
    on Nov 1, 2017
    Contributor
    More actions

    After digging in to it option 1 wasn't really viable because subsequent package builds would fail as the structure it needs would be gone. Option 2 was the most work so I first tried Option 3 and was able to get it working. PR #5291 has been submitted.

    /cc Aditya Patwardhan (@adityapatwardhan)

  7. Travis Plunk (TravisEz13)
    closed this as completedin #5291on Nov 6, 2017
  8. Ilya (iSazonov)
    added
    Resolution-FixedThe issue is fixed.The issue is fixed.
    on Nov 12, 2017
  9. Travis Plunk (TravisEz13)
    added a commit that references this issue on Jul 20, 2018

    Fix macOS launcher app to allow release and preview versions. (#7306)

    d9c570a
  10. Thatgfsj (Thatgfsj)
    added a commit that references this issue on Aug 6, 2026

    Fix macOS launcher app to allow release and preview versions. (PowerS…

    0b616d0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Resolution-FixedThe issue is fixed.The issue is fixed.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402520 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.