Skip to content

[Intelrdfpmathlib] Fix post build error#23286

Merged
ras0219-msft merged 8 commits intomicrosoft:masterfrom
Thomas1664:intelrdfpmathlib
Mar 2, 2022
Merged

[Intelrdfpmathlib] Fix post build error#23286
ras0219-msft merged 8 commits intomicrosoft:masterfrom
Thomas1664:intelrdfpmathlib

Conversation

@Thomas1664
Copy link
Contributor

Describe the pull request

  • What does your PR fix?

    In the latest release I noticed that post build checks failed for intelrdfpmathlib:
2022-02-24T12:59:41.7575882Z -- Performing post-build validation
2022-02-24T12:59:41.7576538Z The software license must be available at ${CURRENT_PACKAGES_DIR}/share/intelrdfpmathlib/copyright
2022-02-24T12:59:41.7578623Z Found 1 post-build check problem(s). To submit these ports to curated catalogs, please first correct the portfile:
2022-02-24T12:59:41.7579258Z     /agent/_work/1/s/ports/intelrdfpmathlib/portfile.cmake
2022-02-24T12:59:41.7579834Z -- Performing post-build validation done
2022-02-24T12:59:41.7761848Z Error: Building package intelrdfpmathlib:x64-linux failed with: POST_BUILD_CHECKS_FAILED
  • Which triplets are supported/not supported? Have you updated the CI baseline?

    unchanged; intelrdfpmathlib is listed in CI baseline

  • Does your PR follow the maintainer guide?

    Yes

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

    Yes

Copy link

@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.

You have modified or added at least one vcpkg.json where a "license" field is missing.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/intelrdfpmathlib/vcpkg.json

Valid values for the license field can be found in the documentation

Copy link

@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.

You have modified or added at least one vcpkg.json where a "license" field is missing.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/intelrdfpmathlib/vcpkg.json

Valid values for the license field can be found in the documentation

@Cheney-W Cheney-W added the category:port-bug The issue is with a library, which is something the port should already support label Feb 28, 2022
Copy link

@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.

You have modified or added at least one vcpkg.json where a "license" field is missing.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/intelrdfpmathlib/vcpkg.json

Valid values for the license field can be found in the documentation

@Cheney-W
Copy link
Contributor

Waiting for CI issue fixed.

Copy link

@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.

You have modified or added at least one vcpkg.json where a "license" field is missing.

Details

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/intelrdfpmathlib/vcpkg.json

Valid values for the license field can be found in the documentation

@Cheney-W Cheney-W added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Mar 1, 2022
@ras0219-msft ras0219-msft merged commit 5326d84 into microsoft:master Mar 2, 2022
@ras0219-msft
Copy link
Contributor

LGTM, thanks for the fix!

Jimmy-Hu added a commit to Jimmy-Hu/vcpkg that referenced this pull request Mar 2, 2022
[Intelrdfpmathlib] Fix post build error (microsoft#23286)
@Thomas1664 Thomas1664 deleted the intelrdfpmathlib branch March 2, 2022 05:33
ekilmer added a commit to ekilmer/vcpkg that referenced this pull request Mar 2, 2022
* master: (108 commits)
  [calceph] update to version 3.5.1 (microsoft#23325)
  [SpeexDSP] Missing windows definitions (microsoft#23296)
  [triton] Fix usage (microsoft#23250)
  [vcpkg_configure_make] Report warning when unresolved arguments are passed in (microsoft#22567)
  [openmvg] no absolute paths (microsoft#22342)
  [python2] no absolute paths (microsoft#22329)
  [libcpplocate] New port (microsoft#23173)
  [Intelrdfpmathlib] Fix post build error (microsoft#23286)
  [docs] Update maintainer guide to include CMAKE_REQUIRE_FIND_PACKAGE. (microsoft#23321)
  [new-port-template] Remove deprecated cmake (microsoft#23283)
  [xtensor] Update to 0.24.0 (microsoft#23238)
  [pcl/soil2] Disable optimizations for individual function (microsoft#23252)
  [cnats] Add new port (C client for the NATS messaging system). (microsoft#23258)
  [nanobench] Update to 4.3.6 (microsoft#23262)
  [elfio] Update elfio to 3.10 (microsoft#23263)
  [tomlplusplus] Update to 3.0.1 (microsoft#23268)
  [shaderc] Fix CMake export target files (microsoft#23269)
  [PCL] Fix namespace and  VS2022 error C3052 (microsoft#23273)
  [libressl] update to 3.4.2 (microsoft#23276)
  [asio-grpc] Update to 1.4.0 (microsoft#23277)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants