Santandersecurityresearch/DrHeader

Build Errors , missing yaml.h

Closed this issue · 2 comments

  • drHEADer version: 0.1.0
  • Python version: all
  • Operating System: ubuntu

Description

When building a distribution of Dr Header there is an error building the YAML dependencies.

Running PyYAML-5.1.2/setup.py -q bdist_egg --dist-dir /var/folders/jx/c1xk5bf5693_6kdwh7ypcgd80000gn/T/easy_install-comdawnw/PyYAML-5.1.2/egg-dist-tmp-lba4j7gs
In file included from ext/_yaml.c:596:
ext/_yaml.h:2:10: fatal error: 'yaml.h' file not found
#include <yaml.h>## What I Did

This looks like its just missing the libyaml-dev package which will need apt-get installing in the build scripts. We aren't building eggs anymore so the setup command has changed , however that won't stop this issue.

This one turns out to be a tricky to resolve in our github actions , to fix it we would need to run the apt package manager as root which is not possible.

It seems that this error is not fatal and that it is just one attempt to get that file that has failed and another attempt seems to succeed. This often happens with Python modules that an OS depends on , the OS vendor adjusts the package manager it ships to ensure its dependencies don't get altered.

We could possibly work around this by doing builds in docker containers in future if it causes problems , for now it doesn't seem to. Will leave this open for now so it stays visible.

This issue has now gone away because it seems github updated their system images and they now include libyaml-dev .