|Summary:||Mono 4.4.1 OSX package doesn't clean up old symlinks from /usr/local/bin|
|Product:||[Mono] Runtime||Reporter:||Alexander Köplinger [MSFT] <alkpli>|
|Component:||packaging||Assignee:||Alexis Christoforides <lexas>|
|Severity:||normal||CC:||benjamin.beckley, david, kyle.white, luis.aguilera, mono-bugs+mono, mono-bugs+runtime, peter.collins, saurabhd, shrutis|
|Target Milestone:||4.8.0 (C9)|
|Tags:||C8SR1 potentialC8SR2||Is this bug a regression?:||No|
|Last known good build:|
Description Alexander Köplinger [MSFT] 2016-07-05 19:12:49 UTC
Since https://github.com/xamarin/bockbuild/commit/4bf7d2ee28d5fd1c6c9c4cb0231f0cef8a8ac859 we moved to using /etc/path.d for adding Mono to the PATH on OSX instead of putting the symlinks in /usr/local/bin. However, the installer doesn't clean the old symlinks up. We should try to delete those in postinstall (with appropriate safeguard measures should Apple decide to further lock down /usr/local).
Comment 1 Luis Aguilera 2016-09-12 20:15:06 UTC
since C8 is now closed, and is shipping this week, I will move this but to the C8SR1 milestone. We'll continue working on the issue seeking it's resolution as soon as possible.
Comment 2 Alexis Christoforides 2016-12-19 14:48:43 UTC
Fixed in mono/bockbuild master and xamarin/bockbuild mono-4.8.0-branch.
Comment 3 Alexander Köplinger [MSFT] 2016-12-19 17:56:16 UTC
@Alexis I don't see a commit in mono/bockbuild master, did you forget to push?
Comment 4 Saurabh 2016-12-23 12:18:27 UTC
I have checked this with latest Cycle9 Mono 220.127.116.113 and observed that on location /etc/path.d I am seeing mono-commands and on location /usr/local/bin, I am not seeing any mono. This is the screencast for the same: https://www.screencast.com/t/3545W4RLSPb Please let me know if there is anything else to verify this Issue?
Comment 5 Shruti 2017-01-10 08:21:58 UTC
@Alexander, It would be great help If you put your comments for Comment4. If the Saurabh Scenario to verify this issue is correct. Then I think we are good to change the status of this bug as Verified.
Comment 6 Alexander Köplinger [MSFT] 2017-01-10 15:46:42 UTC
@Shruti yes, it looks like this is working as intended in Mono 4.8. However I still don't see the relevant fix applied to the master branch in bockbuild. I think it doesn't make sense to hold up verifying this bug for that though, I'll talk to Alexis separately. Changing status to Verified, thanks.