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

Upgrade from TagLib 1 to TagLib 2 #147

Merged
merged 14 commits into from
Oct 26, 2024
Merged

Upgrade from TagLib 1 to TagLib 2 #147

merged 14 commits into from
Oct 26, 2024

Conversation

jacobvosmaer
Copy link
Collaborator

This updates taglib-ruby to work with TagLib 2. This breaks compatibility with TagLib 1.

This change does not introduce new features; we are only making sure that taglib-ruby can compile against TagLib 2.

We are doing this in preparation for releasing taglib-ruby 2.

jacobvosmaer and others added 14 commits March 24, 2024 14:23
This solves a problem brought up in
#129 (comment).
If multiple versions of TagLib are installed and the user sets
TAGLIB_DIR, then we should use only TAGLIB_DIR and not also check some
default directories (like /usr/local).
Ignore default directories when TAGLIB_DIR is set
This will help with the switch to TagLib 2 because that version has a
dependency in a Git submodule.
TagLib 2: ignore MP4 const iterator begin and end
Copy link
Owner

@robinst robinst left a comment

Choose a reason for hiding this comment

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

Thanks @jacobvosmaer!

%include <taglib/tfile.h>

%ignore TagLib::FileRef::properties;
Copy link
Owner

Choose a reason for hiding this comment

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

I've created an issue to support properties and complexProperties:

@jacobvosmaer jacobvosmaer merged commit ddbd238 into main Oct 26, 2024
4 checks passed
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.

3 participants