Skip to content

Fix missing Python bindings for OIIO::getattribute#1290

Merged
lgritz merged 1 commit intoAcademySoftwareFoundation:masterfrom
lgritz:lg-python
Dec 16, 2015
Merged

Fix missing Python bindings for OIIO::getattribute#1290
lgritz merged 1 commit intoAcademySoftwareFoundation:masterfrom
lgritz:lg-python

Conversation

@lgritz
Copy link
Collaborator

@lgritz lgritz commented Dec 15, 2015

This is embarrassing: somehow we never added the global getattribute() calls to the Python bindings.

Also notice that the Python bindings for ImageSpec::getattribute were incorrectly bound to "get_attribute". So add "getattribute" and consider "get_attribute" a deprecated synonym.

I will backport this to 1.6, despite being at RC stage, since this is an obvious bug.

Also notice that the Python bindings for ImageSpec::getattribute
were incorrectly bound to "get_attribute". So add "getattribute"
and consider "get_attribute" a deprecated synonym.
@iwootten
Copy link

Thanks for the swift patch for this @lgritz!

@lgritz
Copy link
Collaborator Author

lgritz commented Dec 16, 2015

@iwootten Do you work from master or a release branch? If the latter, which one?

@iwootten
Copy link

I'm using the 1.5.21 release. It looks like some stuff has gone into RB-1.5 since then, so I guess that means I'm a couple of commits back on RB-1.5.

@lgritz
Copy link
Collaborator Author

lgritz commented Dec 16, 2015

We are a few days from making 1.6 be the official release. I don't suppose you are in a position to switch to 1.6? Or do I need to backport to 1.5 as well?

@lgritz lgritz merged commit 7150f2d into AcademySoftwareFoundation:master Dec 16, 2015
@lgritz
Copy link
Collaborator Author

lgritz commented Dec 16, 2015

Merged into master, RB-1.6, RB-1.5.

@lgritz lgritz deleted the lg-python branch December 16, 2015 22:02
@iwootten
Copy link

Thanks for this! Successfully using it on the 1.5.22 release.

GerHobbelt pushed a commit to GerHobbelt/oiio that referenced this pull request Dec 10, 2024
…oftwareFoundation#1290)

Signed-off-by: Bernard Lefebvre <bernard.lefebvre@autodesk.com>
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