Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

PovlAbrahamsen
Copy link
Contributor

This is a new version of libxaw3dxft, now moved from Sourceforge to https://github.com/DaveFlater/libXaw3dXft. The major change here is that the library has been renamed from libXaw3d.8.dylib to libXaw3dxft.8.dylib. I have created a copy with the old name (and then update the install name). Hopefully this should work for future updates, while maintaining backward compatibility. Tested on MacOS 15.6 Intel and Apple Silicon. Maintainer is @akhansen

@PovlAbrahamsen PovlAbrahamsen mentioned this pull request Aug 3, 2025
@dmacks
Copy link
Member

dmacks commented Aug 4, 2025

This is a clever approach to maintaining compatibility when our original package and its Shlibs filename were out-of-sync and the new upstream version gets them into sync. Minor nit is that you'd also want to symlink the new .pc to be available at the old one's name.

However, the new version actually appears to be not-backwards-compatable. I see multiple public symbols in the old .dylib and .h that are not present in the new one. Given that, and our existing package's naming oddity, it is probably simpler to make a clean break, with a brand new package-name. The old one doesn't include the .dylib ABI-version in its name, so "libxaw3dxft8" (or "libxaw3dxft.8") and associated -shlibs are available. That requires no renaming, linking, or other work on your part to maintain compat.

@PovlAbrahamsen
Copy link
Contributor Author

That makes sense. I'll try to write an updated, simplified package this evening, and will update this PR when ready.

@PovlAbrahamsen PovlAbrahamsen changed the title libxaw3dxft: v1.6.4 libxaw3dxft8-shlibs: v1.6.4 Aug 4, 2025
@PovlAbrahamsen
Copy link
Contributor Author

I've now updated this PR to add a new libxaw3dxft8-shlibs (and -dev) package. These should be able to coexist with the previous xaw3d/xaw3dxft packages. No patches have been applied.

@dmacks
Copy link
Member

dmacks commented Aug 10, 2025

Looks good. Thanks for working on this! I merged it manually (squashed the two commits, but left you as the author, in 4a1f71e), and then made a few dep tweaks to both the old and new lib packages in a separate commit. So this ticket is all done.

@dmacks dmacks closed this Aug 10, 2025
@PovlAbrahamsen PovlAbrahamsen deleted the libxaw3dxft branch August 10, 2025 15:48
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