Skip to content

fixed #13934 - removed deprecated platforms unix32-unsigned and unix64-unsigned#7721

Merged
firewave merged 1 commit intodanmar:mainfrom
firewave:unx-unsigned
Oct 3, 2025
Merged

fixed #13934 - removed deprecated platforms unix32-unsigned and unix64-unsigned#7721
firewave merged 1 commit intodanmar:mainfrom
firewave:unx-unsigned

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Aug 6, 2025

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Aug 6, 2025

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Aug 6, 2025

Still needs code which removes these files when updating an installation on Windows and during the copying step. The latter is necessary so you do not have to manually delete the files for the tests to succeed.

@firewave firewave force-pushed the unx-unsigned branch 5 times, most recently from 73b7870 to 4586f48 Compare September 15, 2025 07:14
@firewave
Copy link
Copy Markdown
Collaborator Author

Successful release build: https://github.com/danmar/cppcheck/actions/runs/17725419174

@firewave firewave marked this pull request as ready for review September 15, 2025 07:39
@sonarqubecloud
Copy link
Copy Markdown

Comment on lines -152 to -153
<File Id='unix32unsigned.xml' Name='unix32-unsigned.xml' Source='$(var.PtfsDir)\unix32-unsigned.xml' />
<File Id='unix64unsigned.xml' Name='unix64-unsigned.xml' Source='$(var.PtfsDir)\unix64-unsigned.xml' />
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The installer will automatically remove these files on an update.

@firewave firewave merged commit e3c8bdc into danmar:main Oct 3, 2025
61 checks passed
@firewave firewave deleted the unx-unsigned branch October 3, 2025 09:06
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