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

gcc: disable libssp #13528

Merged
merged 1 commit into from Oct 11, 2022
Merged

gcc: disable libssp #13528

merged 1 commit into from Oct 11, 2022

Conversation

MehdiChinoune
Copy link
Collaborator

No description provided.

@lazka lazka merged commit 90ddfec into msys2:master Oct 11, 2022
@MehdiChinoune MehdiChinoune deleted the gcc-no-libssp branch October 11, 2022 10:19
@dscho
Copy link
Contributor

dscho commented Oct 12, 2022

Hmm. This might have made sense from the point of view that everything inside MSYS2 is built and packaged by MSYS2. But what about custom-built software? (Or for that matter, Git for Windows?)

It would probably have made more sense to keep a working libssp-0.dll in place, at least for a while, maybe with a DllMain() that warns about that library going away. As it is, there were a scant 3 days grace period for downstream users to adapt and plan ahead, 3 days during which some people were on vacation (me, for example, until I got pulled back ☹️).

@jeremyd2019
Copy link
Member

#13401 (comment) I guess I wasn't explicit that included 3rd party binaries, not just those in packages.

@lazka
Copy link
Member

lazka commented Oct 13, 2022

Sorry for the inconvenience, reverted now (in git)

@dscho
Copy link
Contributor

dscho commented Oct 13, 2022

It's not a big deal, but a good opportunity to keep in mind how far-reaching seemingly innocuous changes can be.

@lazka
Copy link
Member

lazka commented Oct 13, 2022

Reverted in the repo now

@lazka
Copy link
Member

lazka commented May 5, 2023

It's been 7 months now, and I checked that both git-sdk-64 and git-sdk-32 no longer link anything against libssp. So I think we can drop it now.

@lazka
Copy link
Member

lazka commented May 6, 2023

PR for removal: #17080

@lazka
Copy link
Member

lazka commented May 7, 2023

removed now

@dscho
Copy link
Contributor

dscho commented May 8, 2023

It's been 7 months now, and I checked that both git-sdk-64 and git-sdk-32 no longer link anything against libssp. So I think we can drop it now.

Thank you for your diligence. Indeed, we do not need this anymore. FWIW we worked around it very easily, via git-for-windows/git-sdk-64@706bd79

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

Successfully merging this pull request may close these issues.

None yet

4 participants