Skip to content

[openexr] Update to 3.1.5#24249

Closed
Thomas1664 wants to merge 60 commits intomicrosoft:masterfrom
Thomas1664:openexr
Closed

[openexr] Update to 3.1.5#24249
Thomas1664 wants to merge 60 commits intomicrosoft:masterfrom
Thomas1664:openexr

Conversation

@Thomas1664
Copy link
Copy Markdown
Contributor

Describe the pull request

  • What does your PR fix?

    Fixes #...
    Note: Duplicate of [openexr] Update to 3.1.4 #20957

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    <all / linux, windows, ...>, <Yes/No>

  • Does your PR follow the maintainer guide?

    Your answer

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    <Yes / I am still working on this PR>

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for osg but no changes to version or port version.
-- Version: 3.6.5#12
-- Old SHA: f438ec006b58ffd222b0f3bdce8d05b21611204e
-- New SHA: d1ca6a4010851c13c831ee1b215d3f93de453017
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
Error: Local changes detected for openexr but no changes to version or port version.
-- Version: 3.1.5
-- Old SHA: 428dc4ee11bf76291aef9d77936dc64be5d987ce
-- New SHA: 9073168a97cccf0f6794a572699c60d319372288
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

You have modified or added at least one portfile where deprecated functions are used.

Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/osg/portfile.cmake

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for osg but no changes to version or port version.
-- Version: 3.6.5#12
-- Old SHA: f438ec006b58ffd222b0f3bdce8d05b21611204e
-- New SHA: d1ca6a4010851c13c831ee1b215d3f93de453017
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

You have modified or added at least one portfile where deprecated functions are used.

Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/osg/portfile.cmake

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for osg but no changes to version or port version.
-- Version: 3.6.5#12
-- Old SHA: f438ec006b58ffd222b0f3bdce8d05b21611204e
-- New SHA: d1ca6a4010851c13c831ee1b215d3f93de453017
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

You have modified or added at least one portfile where deprecated functions are used.

Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/osg/portfile.cmake

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for osg but no changes to version or port version.
-- Version: 3.6.5#12
-- Old SHA: f438ec006b58ffd222b0f3bdce8d05b21611204e
-- New SHA: d4a1112641f525f5f5f410ce1e2959e592b771b7
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

You have modified or added at least one portfile where deprecated functions are used.

Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/osg/portfile.cmake

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for osg but no changes to version or port version.
-- Version: 3.6.5#12
-- Old SHA: f438ec006b58ffd222b0f3bdce8d05b21611204e
-- New SHA: d4a1112641f525f5f5f410ce1e2959e592b771b7
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

You have modified or added at least one portfile where deprecated functions are used.

Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/osg/portfile.cmake

@Adela0814 Adela0814 added the category:port-update The issue is with a library, which is requesting update new revision label Apr 20, 2022
@dg0yt
Copy link
Copy Markdown
Contributor

dg0yt commented Apr 20, 2022

I don't see an official FindOpenEXR.cmake in CMake. So what is the wrapper targeting at? Each downstream may have it is own variant of the find module. Differences must be handled in downstream ports. Port openexr must not accumulate dependencies on downstream find module implementations.

@dg0yt
Copy link
Copy Markdown
Contributor

dg0yt commented Apr 20, 2022

To limit downstream patching, would be a transitional option to:

  • add forwarding headers from openexr to imath (in particular for half.h)
  • add a dependency on imath config to openexr config?

@dg0yt
Copy link
Copy Markdown
Contributor

dg0yt commented Apr 20, 2022

To limit downstream patching, would be a transitional option to:

  • add forwarding headers from openexr to imath (in particular for half.h)

  • add a dependency on imath config to openexr config?

... perhaps implemented behind an opt-in openexr[imath-transition] feature.

@Thomas1664
Copy link
Copy Markdown
Contributor Author

@dg0yt Seems like you're more familiar with this than me. Feel free to open your own PR to update openexr. Honestly, I don't want to work on another OpenSSL-like patch thing.

@Adela0814
Copy link
Copy Markdown
Contributor

@Thomas1664 Is work still being done for this PR?

@Thomas1664
Copy link
Copy Markdown
Contributor Author

@Thomas1664 Is work still being done for this PR?

No. I don't have time for this.

@Adela0814
Copy link
Copy Markdown
Contributor

Closing this PR, please reopen if you continue this work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-update The issue is with a library, which is requesting update new revision

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants