Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

graphviz is using host includes/libs [hardknott], when tcl is installed on the host #407

Open
kranzo opened this issue Aug 7, 2021 · 3 comments

Comments

@kranzo
Copy link

kranzo commented Aug 7, 2021

if build in crops(ubuntu-20.04) everything works properly

on my baremetal workstation (ubuntu-20.04) tlc8.6 is installed, resulting into QA-Error because host paths are included.

removing this patch leads to a error free compilation: https://github.com/openembedded/meta-openembedded/blob/9b23e1f76b76a43ea87e6364c23f29a21da81e61/meta-oe/recipes-graphics/graphviz/graphviz/0001-Set-use_tcl-to-be-empty-string-if-tcl-is-disabled.patch

so my question is why would i configure tcl if tcl is explicitly disabled? Because setting the string to "" is actually the default (use_tcl) config. I feel like the patch is not doing what is said in the commit-msg

@michalfita
Copy link

On Ubuntu 22.04 with Yocto Honister (brought in by PetaLinux v2022.1) graphviz was failing for me with:

/usr/lib//tcl8.6/tclConfig.sh: line 2: dpkg-architecture: command not found

Removing this patch as above issue mentions solves the problem for me.

The recipe in use is components/yocto/layers/meta-openembedded/meta-oe/recipes-graphics/graphviz/graphviz_2.44.1.bb.

Thanks, @kranzo.

@kraj
Copy link
Contributor

kraj commented Jan 10, 2023

@kranzo @michalfita thanks for report. honister is EOL but if you can reproduce it with master or kirkstone, that would be awesome. I am using archlinux host and has tclsh8.6 installed but I do not see this error on master branch

@Youras
Copy link

Youras commented Apr 11, 2024

Somehow removing this patch, didn't solve the error. But here is a dirty fix:

The cause of the error is sourcing a tcl config script located in /usr/lib/tcl8.6/tclConfig.sh as the error prompt says

#! /bin/sh
. /usr/lib/`dpkg-architecture -qDEB_HOST_MULTIARCH`/tcl8.6/tclConfig.sh

Now dpkg-architecture -qDEB_HOST_MULTIARCH can be resolved by simply running this part in a console and replacing this command by the prompt of the console.
In my case the prompt is x86_64-linux-gnu. Thus the final file may look like this:

#! /bin/sh
. /usr/lib/`dpkg-architecture -qDEB_HOST_MULTIARCH`/tcl8.6/tclConfig.sh

Now building should work fluently :)

kraj pushed a commit to YoeDistro/meta-openembedded that referenced this issue Jun 12, 2024
Changelog:
===========
- Fixes openembedded#409: IndexError on empty strong mark.
- Fix openembedded#332: Insert at most one space for multiple emphasis
- Feature openembedded#318: Make padded tables more similar to pandoc's pipe_tables.
- Add support for Python 3.9.
- Fix extra line breaks inside html link text (between '[' and ']')
- Fix openembedded#344: indent <ul> inside <ol> three spaces instead of two to comply with CommonMark, GFM, etc.
- Fix openembedded#324: unnecessary spaces around <b>, <em>, and strike tags.
- Don't wrap tables by default and add a --wrap-tables config option.
- Feature openembedded#198: Ignore <p> tags inside table rows.
- Don't wrap tables by default and add a --wrap-tables config option
- Remove support for Python <= 3.5. Now requires Python 3.6+.
- Support for Python 3.10+.
- Fix openembedded#320 padding empty tables and tables with no </tr> tags.
- Add ignore_mailto_links config option to ignore mailto: style links.
- Feature openembedded#407: Support the superscript and subscript tags.
- Fix openembedded#373: \n inside text of a Markdown link.
- Feature openembedded#406: Improve support for null atttibute values.

Signed-off-by: Wang Mingyu <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
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

No branches or pull requests

4 participants