Skip to content

Conversation

@regro-cf-autotick-bot
Copy link
Contributor

It is very likely that the current package version for this feedstock is out of date.

Checklist before merging this PR:

  • Dependencies have been updated if changed: see upstream
  • Tests have passed
  • Updated license if changed and license_file is packaged

Information about this PR:

  1. Feel free to push to the bot's branch to update this PR if needed.
  2. The bot will almost always only open one PR per version.
  3. The bot will stop issuing PRs if more than 3 version bump PRs generated by the bot are open. If you don't want to package a particular version please close the PR.
  4. If you want these PRs to be merged automatically, make an issue with @conda-forge-admin,please add bot automerge in the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.
  5. If this PR was opened in error or needs to be updated please add the bot-rerun label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase @conda-forge-admin, please rerun bot in a PR comment to have the conda-forge-admin add it for you.

Pending Dependency Version Updates

Here is a list of all the pending dependency version updates for this repo. Please double check all dependencies before merging.

Name Upstream Version Current Version
gz-math 8_8.2.0 Anaconda-Server Badge

This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/16539347737 - please use this URL for debugging.

@conda-forge-admin
Copy link
Contributor

conda-forge-admin commented Jul 26, 2025

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/18164920433. Examine the logs at this URL for more detail.

@conda-forge-admin
Copy link
Contributor

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

  • linter: passed
  • azure: failed

Thus the PR was not passing and not merged.

@traversaro
Copy link
Contributor

@conda-forge-admin please rerender

@conda-forge-admin
Copy link
Contributor

Hi! This is the friendly conda-forge automerge bot!

It appears that not all commits to this PR were made by the bot. Thus this PR is not being automatically merged. Please add the automerge label again (or ask a maintainer to do so) if you'd like to enable automerge again!

@conda-forge-admin
Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/17216169634. Examine the logs at this URL for more detail.

@traversaro
Copy link
Contributor

macOS error:

/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include/OGRE-Next/OgreUTFString.h:235:13: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
  235 |             dstring::iterator mIter;
      |             ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/__fwd/string.h:23:29: note: template is declared here
   23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
      |                             ^
In file included from /Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/work/ogre2/src/Ogre2FrustumVisual.cc:31:
In file included from /Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/string:647:
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/string_view:300:42: error: implicit instantiation of undefined template 'std::char_traits<unsigned short>'
  300 |   static_assert(is_same<_CharT, typename traits_type::char_type>::value,
      |                                          ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/__type_traits/is_convertible.h:22:99: note: in instantiation of template class 'std::basic_string_view<unsigned short>' requested here
   22 | struct _LIBCPP_TEMPLATE_VIS is_convertible : public integral_constant<bool, __is_convertible(_T1, _T2)> {};
      |                                                                                                   ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/string:745:29: note: in instantiation of template class 'std::is_convertible<const unsigned short *const &, std::basic_string_view<unsigned short>>' requested here
  745 |     : public _BoolConstant< is_convertible<const _Tp&, basic_string_view<_CharT, _Traits> >::value &&
      |                             ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/string:1542:38: note: in instantiation of template class 'std::__can_be_converted_to_string_view<unsigned short, std::char_traits<unsigned short>, const unsigned short *>' requested here
 1542 |   template <class _Tp, __enable_if_t<__can_be_converted_to_string_view<_CharT, _Traits, _Tp>::value, int> = 0>
      |                                      ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/string:1544:3: note: while substituting prior template arguments into non-type template parameter [with _Tp = const code_point *]
 1544 |   insert(size_type __pos1, const _Tp& __t) {
      |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 1545 |     __self_view __sv = __t;
      |     ~~~~~~~~~~~~~~~~~~~~~~~
 1546 |     return insert(__pos1, __sv.data(), __sv.size());
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 1547 |   }
      |   ~
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include/OGRE-Next/OgreUTFString.h:662:19: note: while substituting deduced template arguments into function template 'insert' [with _Tp = const code_point *, $1 = (no value)]
  662 |             mData.insert( index, str );
      |                   ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/__fwd/string.h:23:29: note: template is declared here
   23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
      |                             ^
In file included from /Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/work/ogre2/src/Ogre2FrustumVisual.cc:31:
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/string:3087:29: error: implicit instantiation of undefined template 'std::char_traits<unsigned short>'
 3087 |   return insert(__pos, __s, traits_type::length(__s));
      |                             ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/include/OGRE-Next/OgreUTFString.h:662:19: note: in instantiation of member function 'std::basic_string<unsigned short>::insert' requested here
  662 |             mData.insert( index, str );
      |                   ^
/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/_build_env/bin/../include/c++/v1/__fwd/string.h:23:29: note: template is declared here
   23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
      |                             ^
3 errors generated.
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
  File "/Users/runner/miniforge3/lib/python3.12/site-packages/conda_build/build.py", line 1797, in bundle_conda
    utils.check_call_env(
  File "/Users/runner/miniforge3/lib/python3.12/site-packages/conda_build/utils.py", line 411, in check_call_env
    return _func_defaulting_env_to_os_environ("call", *popenargs, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/runner/miniforge3/lib/python3.12/site-packages/conda_build/utils.py", line 387, in _func_defaulting_env_to_os_environ
    raise subprocess.CalledProcessError(proc.returncode, _args)
subprocess.CalledProcessError: Command '['/bin/bash', '-e', '/Users/runner/miniforge3/conda-bld/gz-rendering9_1756143500398/work/build_cxx.sh']' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:

probably due to update in compiler.

@j-rivero
Copy link

I think that the problem here is a change in the libc++ , concretely in 19 version;

The base template for std::char_traits has been removed in LLVM 19. If you are using std::char_traits with types other than char, wchar_t, char8_t, char16_t, char32_t or a custom character type for which you specialized std::char_traits, your code will stop working. The Standard does not mandate that a base template is provided, and such a base template is bound to be incorrect for some types, which could currently cause unexpected behavior while going undetected.

Line 235 in OgreUTFString seems to be triggering the error by a template expansion over dstring that uses a uint32.

Not sure about the best patch here, I'm going to report it upstream.

Updated conda build configuration to use clang 18.
@traversaro
Copy link
Contributor

Thanks a lot @j-rivero and @iche033 for the follow up on this! As it is possible to see in llvm/llvm-project@aeecef0, all the removed functions were inline, so I think it is safe to just keep using clang 18 for the time being, while keeping the generated binaries compatible with newer libcxx versions. However, I wonder if before gz-rendering10 (where we are going to have an ABI bump anyhow) could it make sense to backport https://github.com/gazebosim/gz-rendering/pull/1177/files and set the CMake option USE_OGRE2_OVERLAY to OFF so we can use newer clang version across all the life of the gz-rendering10 release?

@traversaro
Copy link
Contributor

@conda-forge-admin please rerender

@j-rivero
Copy link

j-rivero commented Oct 1, 2025

However, I wonder if before gz-rendering10 (where we are going to have an ABI bump anyhow) could it make sense to backport https://github.com/gazebosim/gz-rendering/pull/1177/files and set the CMake option USE_OGRE2_OVERLAY to OFF so we can use newer clang version across all the life of the gz-rendering10 release?

That makes sense to me yes. The only problem I had with gazebosim/gz-rendering#1177 is that the ABI checker is complaining for the OFF default option, although I'm not sure that it is correct. In the Jetty bump to 10, I would say let's backport gazebosim/gz-rendering#1177 here and enable the option from the beginning as you suggest, even if the ABI checker is true about ABI problems in default OFF we always be using the ON version here so ABI is preserve. This has the risk that we discover some runtime problem with the lack of Overlay but I think that it is a risk that can take.

In the next major (11) there will be no problem if the gazebosim/gz-rendering#1175 goes well.

@traversaro
Copy link
Contributor

The only problem I had with gazebosim/gz-rendering#1177 is that the ABI checker is complaining for the OFF default option, although I'm not sure that it is correct.

I think there was a missing assignment of a CMake variable, see gazebosim/gz-rendering#1177 (comment) and gazebosim/gz-rendering#1177 (comment) .

@traversaro
Copy link
Contributor

@conda-forge-admin please rerender

@traversaro traversaro added the automerge Merge the PR when CI passes label Oct 1, 2025
@traversaro traversaro merged commit 9fa7d35 into conda-forge:main Oct 1, 2025
7 of 9 checks passed
@regro-cf-autotick-bot regro-cf-autotick-bot deleted the 9.3.0_hcc5775 branch October 1, 2025 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the PR when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants