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

Fix a couple of build errors #103

Merged
merged 1 commit into from
Jun 24, 2024
Merged

Conversation

JonathanAquino-NextRoll
Copy link
Contributor

Fix a couple of build errors I saw while running BatchiePatchie locally today.

@@ -1,11 +1,12 @@
FROM node:10.15.3-stretch
FROM node:10.16.3-buster
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This upgrade fixes the following error:

#5 4.954 Err:10 http://deb.debian.org/debian stretch/main arm64 Packages
#5 4.954   404  Not Found
#5 4.982 Ign:11 http://deb.debian.org/debian stretch-updates/main all Packages
#5 5.012 Err:12 http://deb.debian.org/debian stretch-updates/main arm64 Packages
#5 5.012   404  Not Found
#5 5.035 Reading package lists...
#5 5.048 W: The repository 'http://security.debian.org/debian-security stretch/updates Release' does not have a Release file.
#5 5.048 W: The repository 'http://deb.debian.org/debian stretch Release' does not have a Release file.
#5 5.048 W: The repository 'http://deb.debian.org/debian stretch-updates Release' does not have a Release file.
#5 5.048 E: Failed to fetch http://security.debian.org/debian-security/dists/stretch/updates/main/binary-arm64/Packages  404  Not Found
#5 5.048 E: Failed to fetch http://deb.debian.org/debian/dists/stretch/main/binary-arm64/Packages  404  Not Found
#5 5.048 E: Failed to fetch http://deb.debian.org/debian/dists/stretch-updates/main/binary-arm64/Packages  404  Not Found
#5 5.048 E: Some index files failed to download. They have been ignored, or old ones used instead.
------
executor failed running [/bin/sh -c apt-get update && apt-get install -y     build-essential     python     python-dev     python-pip]: exit code: 100
ERROR: Service 'frontend' failed to build : Build failed


# AWS cli tools
RUN apt-get update && apt-get install -y \
build-essential \
python \
python-dev \
python-pip
RUN pip install PyYAML==5.3.1
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This downgrade fixes the following error. Also see yaml/pyyaml#724.

#6 21.25       File "/tmp/pip-build-env-FfX2NQ/lib/python2.7/site-packages/setuptools/command/py36compat.py", line 36, in add_defaults
#6 21.25         self._add_defaults_ext()
#6 21.25       File "/tmp/pip-build-env-FfX2NQ/lib/python2.7/site-packages/setuptools/command/py36compat.py", line 120, in _add_defaults_ext
#6 21.25         self.filelist.extend(build_ext.get_source_files())
#6 21.25       File "/tmp/pip-install-JOgnpr/PyYAML/setup.py", line 201, in get_source_files
#6 21.25         self.cython_sources(ext.sources, ext)
#6 21.25       File "/usr/lib/python2.7/distutils/cmd.py", line 105, in __getattr__
#6 21.25         raise AttributeError, attr
#6 21.25     AttributeError: cython_sources
#6 21.25
#6 21.25     ----------------------------------------
#6 22.13 Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-JOgnpr/PyYAML/
------
executor failed running [/bin/sh -c pip install awscli]: exit code: 1
ERROR: Service 'frontend' failed to build : Build failed

Copy link

@eagarcia-adroll eagarcia-adroll left a comment

Choose a reason for hiding this comment

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

LGTM

@JonathanAquino-NextRoll JonathanAquino-NextRoll merged commit 15bb17a into master Jun 24, 2024
1 check passed
@JonathanAquino-NextRoll JonathanAquino-NextRoll deleted the fix-build-errors branch June 24, 2024 23:26
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