Skip to content

[REACTOS] Change manifest architecture from X86 to * #7930

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

Merged
merged 1 commit into from
May 5, 2025

Conversation

tkreuzer
Copy link
Contributor

@tkreuzer tkreuzer commented Apr 24, 2025

Purpose

Change manifest architecture from X86 to *

Testbot runs (Filled in by Devs)

@tkreuzer tkreuzer self-assigned this Apr 24, 2025
@tkreuzer tkreuzer requested a review from ThFabba as a code owner April 24, 2025 13:11
@github-actions github-actions bot added the ROSTESTS Label for ROS testcases PRs. label Apr 24, 2025
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
<assemblyIdentity type="win32" name="redirtest2" version="0.2.2.2" processorArchitecture="x86" />
<assemblyIdentity type="win32" name="redirtest2" version="0.2.2.2" processorArchitecture="*" />
<file name="kernel32test_versioned.dll"/>
</assembly>

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

@binarymaster binarymaster added the trivial fix For trivial changes. label Apr 24, 2025
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
<assemblyIdentity type="win32" name="Microsoft.VC80.ATL" version="8.0.50727.4053" processorArchitecture="" publicKeyToken="1fc8b3b9a1e18e3b"/>
<assemblyIdentity type="win32" name="Microsoft.VC80.ATL" version="8.0.50727.4053" processorArchitecture="*" publicKeyToken="1fc8b3b9a1e18e3b"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

There are 5 other occurrences (2 gdiplus and 3 msxml), in case they would apply too:
https://git.reactos.org/?p=reactos.git&a=search&h=HEAD&st=grep&s=processorArchitecture%3D%22%22

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@tkreuzer tkreuzer force-pushed the amd64/manifest-arch branch from 2e370bb to bc2d7e0 Compare May 4, 2025 12:49
@tkreuzer tkreuzer merged commit d1f9e06 into reactos:master May 5, 2025
1 check passed
@tkreuzer tkreuzer deleted the amd64/manifest-arch branch May 5, 2025 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ROSTESTS Label for ROS testcases PRs. trivial fix For trivial changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants