Skip to content

3rdparty(itt): support OpenBSD#26797

Closed
brad0 wants to merge 1 commit intoopencv:4.xfrom
brad0:itt_openbsd_support
Closed

3rdparty(itt): support OpenBSD#26797
brad0 wants to merge 1 commit intoopencv:4.xfrom
brad0:itt_openbsd_support

Conversation

@brad0
Copy link
Copy Markdown
Contributor

@brad0 brad0 commented Jan 19, 2025

Cherry picked from commit e09a700d1e5a7d1b8ad57eac85f69813639a82a2 from upstream.

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

Cherry picked from commit e09a700d1e5a7d1b8ad57eac85f69813639a82a2
from upstream.
@Kumataro
Copy link
Copy Markdown
Contributor

I think e09a700d1e5a7d1b8ad57eac85f69813639a82a2 means intel/ittapi@e09a700 in ittapi github.

OpenCV community don't like to hotfix to 3rd party libraries directly.
So I think it is better to upgrade ittapi from upstream.
Currently bundled ittnotify seems be 9 years before.

https://github.com/opencv/opencv/blob/3effe195cb2cf4b7f98fe5e9b4df55aa4951d706/3rdparty/ittnotify/src/ittnotify/ittnotify_config.h#L241C7-L242C38

/* Replace with snapshot date YYYYMMDD for promotion build. */
#define API_VERSION_BUILD    20151119

@brad0
Copy link
Copy Markdown
Contributor Author

brad0 commented Jan 19, 2025

I think e09a700d1e5a7d1b8ad57eac85f69813639a82a2 means intel/ittapi@e09a700 in ittapi github.

OpenCV community don't like to hotfix to 3rd party libraries directly. So I think it is better to upgrade ittapi from upstream. Currently bundled ittnotify seems be 9 years before.

https://github.com/opencv/opencv/blob/3effe195cb2cf4b7f98fe5e9b4df55aa4951d706/3rdparty/ittnotify/src/ittnotify/ittnotify_config.h#L241C7-L242C38

/* Replace with snapshot date YYYYMMDD for promotion build. */
#define API_VERSION_BUILD    20151119

Yes, that is correct.

Ah, I wasn't sure and noticed it was a really old copy. Also there was already a commit for aarch64 support.

Yes, syncing with upstream would be the better option.

@Kumataro
Copy link
Copy Markdown
Contributor

Yes, syncing with upstream would be the better option.

Thank you for your comment, I created PR #26802 to update latest ittnotify,
I hope it is helpful for you.

@opencv-alalek
Copy link
Copy Markdown
Contributor

Closed in favor of #26802

@brad0 brad0 deleted the itt_openbsd_support branch January 19, 2025 22:32
asmorkalov pushed a commit that referenced this pull request Jan 20, 2025
3rdparty:ittnotify: update to v3.25.4 #26802

Close #26801
See #26797

### Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [ ] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [ ] The feature is well documented and sample code can be built with the project CMake
vrabaud pushed a commit to vrabaud/opencv that referenced this pull request Jan 21, 2025
3rdparty:ittnotify: update to v3.25.4 opencv#26802

Close opencv#26801
See opencv#26797

### Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [ ] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [ ] The feature is well documented and sample code can be built with the project CMake
NanQin555 pushed a commit to NanQin555/opencv that referenced this pull request Feb 24, 2025
3rdparty:ittnotify: update to v3.25.4 opencv#26802

Close opencv#26801
See opencv#26797

### Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [ ] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
      Patch to opencv_extra has the same branch name.
- [ ] The feature is well documented and sample code can be built with the project CMake
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