Skip to content

cligen: fix depends - #30144

Merged
feckert merged 1 commit into
openwrt:masterfrom
AndyChiang888:cligen
Aug 4, 2026
Merged

cligen: fix depends#30144
feckert merged 1 commit into
openwrt:masterfrom
AndyChiang888:cligen

Conversation

@AndyChiang888

Copy link
Copy Markdown
Contributor

fixes: openwrt/openwrt#24247 (comment)

📦 Package Details

Maintainer: @pprindeville

Description:


🧪 Run Testing Details

  • OpenWrt Version: Snapshot
  • OpenWrt Target/Subtarget: x86/64
  • OpenWrt Device: N100

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
    make package/<your-package>/refresh V=s
  • It is structured in a way that it is potentially upstreamable
    (e.g., subject line, commit description, etc.)
    We must try to upstream patches to reduce maintenance burden.

@openwrt-ai openwrt-ai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed 1 new commit; no new issues found.


Generated by Claude Code

@httpstorm

Copy link
Copy Markdown
Contributor

Thank you!

This removes the libxml2 related warnings I see when I run:

 make menuconfig

 WARNING: Makefile 'package/feeds/video/sdl3/Makefile' has a build dependency on 'libwayland', which does not exist
 feeds/packages/net/freeradius3/Config.in:3:error: recursive dependency detected!
 feeds/packages/net/freeradius3/Config.in:3:	choice <choice> contains symbol PACKAGE_freeradius3-common
 tmp/.config-package.in:106904:	symbol PACKAGE_freeradius3-common is selected by PACKAGE_freeradius3
 tmp/.config-package.in:106891:	symbol PACKAGE_freeradius3 depends on FREERADIUS3_OPENSSL
 feeds/packages/net/freeradius3/Config.in:11:	symbol FREERADIUS3_OPENSSL is part of choice <choice>
 For a resolution refer to Documentation/kbuild/kconfig-language.rst
 subsection "Kconfig recursive dependency limitations"

 tmp/.config-package.in:106652:error: recursive dependency detected!
 tmp/.config-package.in:106652:	symbol PACKAGE_nftables-nojson is selected by PACKAGE_nftables-nojson
 For a resolution refer to Documentation/kbuild/kconfig-language.rst
 subsection "Kconfig recursive dependency limitations"
-
-tmp/.config-package.in:60639:error: recursive dependency detected!
-tmp/.config-package.in:60639:	symbol PACKAGE_libcurl is selected by PACKAGE_clixon
-tmp/.config-package.in:138677:	symbol PACKAGE_clixon depends on PACKAGE_libxml2
-tmp/.config-package.in:66013:	symbol PACKAGE_libxml2 is selected by PACKAGE_libintl-full
-tmp/.config-package.in:62740:	symbol PACKAGE_libintl-full is selected by PACKAGE_libidn2
-tmp/.config-package.in:62618:	symbol PACKAGE_libidn2 is selected by PACKAGE_libcurl
-For a resolution refer to Documentation/kbuild/kconfig-language.rst
-subsection "Kconfig recursive dependency limitations"

Any advice or fix for the remaining warnings is welcome.

@AndyChiang888

Copy link
Copy Markdown
Contributor Author

@httpstorm see 7933a6f, #30074

@httpstorm

Copy link
Copy Markdown
Contributor

Thank you! With these fixes in place, the only remaining warning I see is the one related to libwayland.
I think on macOS host, it can be installed using brew install wayland.
I wonder if there is a requirement to have that installed or if it needs to be fixed?

@AndyChiang888

Copy link
Copy Markdown
Contributor Author

openwrt/video@51549da#commitcomment-189963371
I reported this issue on June 23, but no one has fixed it yet.

Comment thread utils/cligen/Makefile
fixes: openwrt/openwrt#24247 (comment)

Signed-off-by: Andy Chiang <AndyChiang_git@outlook.com>
@feckert
feckert merged commit c4f3f8d into openwrt:master Aug 4, 2026
14 checks passed
@feckert

feckert commented Aug 4, 2026

Copy link
Copy Markdown
Member

Thanks merged!

@AndyChiang888
AndyChiang888 deleted the cligen branch August 4, 2026 05:38
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.

5 participants