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

Appveyor build changes #7867

Closed
wsmelton opened this issue Oct 16, 2021 · 8 comments
Closed

Appveyor build changes #7867

wsmelton opened this issue Oct 16, 2021 · 8 comments
Labels
bugs life stale Stale Bot label used to identify issues that are no longer active and bot has closed them

Comments

@wsmelton
Copy link
Member

Verified issue does not already exist?

Yes

What error did you receive?

image

Steps to Reproduce

N/A - this is issue with Appveyor runs

Are you running the latest release?

Yes

Other details or mentions

588e3cb

Changes in this PR bumped the .NET/dotnet SDK to 5.0 which is not supported for our library class file. Our project is not written to compile under 5.0 release right now. (And actually never can because it breaks functionality with PowerShell library for cross-platform support.)

What PowerShell host was used when producing this error

PowerShell Core (pwsh.exe)

PowerShell Host Version

N/A

SQL Server Edition and Build number

N/A

.NET Framework Version

Whatever Appveyor runs 🤷🏻

@wsmelton wsmelton added bugs life triage required New issue that has not been reviewed by maintainers labels Oct 16, 2021
@wsmelton
Copy link
Member Author

This run from a current PR also shows the failure but a different error message:

https://ci.appveyor.com/project/sqlcollaborative/dbatools/builds/41175424/job/4yclemy7kr3vhtty

image

@wsmelton wsmelton removed the triage required New issue that has not been reviewed by maintainers label Oct 17, 2021
@andreasjordan
Copy link
Contributor

What can we do here?

@wsmelton
Copy link
Member Author

How the library is built during Appveyor setup would need to be adjusted.

I'd personally like to move the library project to it's own project on GitHub so we can build and test it separately (using GitHub Actions). Then setup an action to publish it as a formal Nuget. Not an overnight change though 🤷.

@wsmelton
Copy link
Member Author

If I recall from last looking at it the version of dotnet SDK we point at is wrong. Our library does not support dotnet 5.0.

@github-actions
Copy link

🚧🚨 This issue is being marked as stale due to 90 days of inactivity. If you would like this issue to remain open:

  • Verify the issue/bug is reproduced in the latest version of the module
  • Verify the environmental info provided is still accurate
  • Add any additional steps you followed to reproduce if necessary
    🚨🚧
    ⌛️ This issue will be closed in 30 days ⌛️

@github-actions github-actions bot added the stale Stale Bot label used to identify issues that are no longer active and bot has closed them label Mar 22, 2022
@andreasjordan
Copy link
Contributor

@potatoqualitee : Is this something you have to keep in mind for the new library module or 2.0?

@potatoqualitee
Copy link
Member

yep! ty

@andreasjordan
Copy link
Contributor

Looks like this is solved now. Can you confirm, @wsmelton ?

@wsmelton wsmelton closed this as not planned Won't fix, can't repro, duplicate, stale May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugs life stale Stale Bot label used to identify issues that are no longer active and bot has closed them
Projects
None yet
Development

No branches or pull requests

3 participants