Skip to content

Conversation

MateuszGrabuszynski
Copy link
Collaborator

When building RxTxApp on v25.02, there is an issue with the variable names being called swaped_* instead of swapped_*.

Resolution:

for f in $(find .); do sed -i 's/swaped_first_hdr_chunk/swapped_first_hdr_chunk/g' $f; done
for f in $(find .); do sed -i 's/swaped_second_hdr_chunk/swapped_second_hdr_chunk/g' $f; done
for f in $(git status --short | awk '/^ T/{print $NF}'); do git restore $f; done # avoids LF->CRLF conversion

@DawidWesierski4 DawidWesierski4 changed the title Fix: swaped -> swapped for RxTxApp (v25.02) Style: swaped -> swapped for RxTxApp (v25.02) Jul 29, 2025
@DawidWesierski4 DawidWesierski4 changed the title Style: swaped -> swapped for RxTxApp (v25.02) Fix: swaped -> swapped for RxTxApp (v25.02) Jul 29, 2025
Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

Just checked on my end

image

no similar issues observed
This could be potentially mismatched API as we fixed this typo and you could have the newer include files if you only builder the RxTxApp itself
Please check after downgrading the MTL include / library to correct version

@MateuszGrabuszynski
Copy link
Collaborator Author

MateuszGrabuszynski commented Jul 29, 2025

As discussed, hopefully today, I will retry building the v25.02 on a fresh setup in order to confirm/deny the existence of the issue. The machine where I tested it previously, may have had the MTL built from main branch before building from v25.02, which may had caused the mismatch of APIs. Expect update.

@MateuszGrabuszynski
Copy link
Collaborator Author

As suggested, misaligned APIs were the cause of this issue. Closing as not needed.

@DawidWesierski4 DawidWesierski4 deleted the swaped-to-swapped-v25.02 branch August 7, 2025 10:18
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