Skip to content

update armadillo to 15.2.6#9903

Open
fasiondog wants to merge 4 commits intoxmake-io:devfrom
fasiondog:feature/armadillo
Open

update armadillo to 15.2.6#9903
fasiondog wants to merge 4 commits intoxmake-io:devfrom
fasiondog:feature/armadillo

Conversation

@fasiondog
Copy link
Copy Markdown
Contributor

update armadillo to 15.2.6

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds version 15.2.6 to the Armadillo package configuration. Feedback indicates that this version number is likely incorrect for the Armadillo C++ library, where the latest stable release is 14.2.2, and suggests correcting the version and updating the test block to C++17 to accommodate the requirements of newer releases.

set_license("Apache-2.0")

add_urls("http://sourceforge.net/projects/arma/files/armadillo-$(version).tar.xz")
add_versions("15.2.6", "97cb8ef708541f632e861d005a462dd0367240f81ff96f8e63ebbdd75c8ce55f")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

The version 15.2.6 does not exist for the Armadillo C++ linear algebra library. The latest stable release is 14.2.2. It appears this version number may have been confused with another software of the same name. Additionally, Armadillo 14.0 and later require C++17, so the on_test block at line 53 should be updated to use c++17 instead of c++14 to ensure the package can be verified correctly.

    add_versions("14.2.2", "6e10669665163a60e6493913f1ad7290168da4a939c64531370896d7bc623ad1")

@fasiondog
Copy link
Copy Markdown
Contributor Author

这个 mingw(msys32) 和包无关,CI prepare阶段就不行了

@Issues-translate-bot
Copy link
Copy Markdown

Bot detected the issue body's language is not English, translate it automatically.


This mingw(msys32) has nothing to do with the package, not the CI prepare stage.

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