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

Enable dotnetcore for framework #1406

Open
wants to merge 2 commits into
base: release/27.3.5
Choose a base branch
from

Conversation

bec-callow-oct
Copy link
Contributor

Avoid throwing an exception when attempting to run .NET scripts in the .NET Framework build of Calamari. This is for Pre-Deploy scripts on Deploy a Package as this step does not support .NET Core

[SC-99263]

@bec-callow-oct bec-callow-oct marked this pull request as ready for review December 12, 2024 05:30
Copy link
Contributor

@IsaacCalligeros95 IsaacCalligeros95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bec-callow-oct bec-callow-oct enabled auto-merge (squash) December 12, 2024 05:32
@IsaacCalligeros95
Copy link
Contributor

There's a lot of moving parts on this one. I've had a chat with Trent and we'll need to wait for this change to fix the Calamari Consolidation process when bundling a .Net6 package in a netframework build. Without this change we can overwrite dlls and break a lot of other things. This also requires Server changes to handle the reassembly of the calamari bundle which will have to be backported to Server.

The alternative to this is that the deploy a package step as of 2024.4 supports running against the netcore versions of Calamari and netframework will be deprecated entirely. This is behind a feature toggle, but if the customer was willing to wait for 2024.4 they'll be able to fix this issue by just enabling that feature toggle.

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.

2 participants