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

Add LIBSODIUM_MAKE_ARGS environment variable #303

Merged
merged 1 commit into from
Jun 14, 2017

Conversation

uSpike
Copy link
Contributor

@uSpike uSpike commented Jun 13, 2017

By allowing a user to pass LIBSODIUM_MAKE_ARGS, we can decrease the compile time on slow platforms by a considerable amount.

In a multiarch/alpine:armhf-v3.6 docker container:

  • pip install . takes about 10 minutes
  • LIBSODIUM_MAKE_ARGS=-j4 pip install . takes just under 4 minutes

Fixes #302

Copy link
Member

@reaperhulk reaperhulk left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, this looks fine. I'll merge once tests pass!

@codecov
Copy link

codecov bot commented Jun 13, 2017

Codecov Report

Merging #303 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #303      +/-   ##
==========================================
- Coverage   99.86%   99.86%   -0.01%     
==========================================
  Files          36       36              
  Lines        1448     1446       -2     
  Branches       71       71              
==========================================
- Hits         1446     1444       -2     
  Misses          1        1              
  Partials        1        1
Impacted Files Coverage Δ
src/nacl/bindings/utils.py 100% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e454b69...eace11c. Read the comment docs.

@reaperhulk
Copy link
Member

coverage numbers don't make sense so merging :) thanks again

@reaperhulk reaperhulk merged commit 4b65897 into pyca:master Jun 14, 2017
collivier added a commit to collivier/functest that referenced this pull request May 12, 2019
Building functest-core container sometimes reach travis-ci timeout [0].
It decreases the compile time as proposed upstream [1]

[0] https://travis-ci.org/collivier/functest/jobs/531417730
[1] pyca/pynacl#303

Change-Id: Ie6c8e729f76c98c6f8c8e9ca01b2c998081f7ce8
Signed-off-by: Cédric Ollivier <[email protected]>
collivier added a commit to collivier/functest that referenced this pull request May 13, 2019
Building functest-core container sometimes reaches travis-ci timeout
[0]. It decreases the compile time as proposed upstream [1].

[0] https://travis-ci.org/collivier/functest/jobs/531417730
[1] pyca/pynacl#303

Change-Id: Ie6c8e729f76c98c6f8c8e9ca01b2c998081f7ce8
Signed-off-by: Cédric Ollivier <[email protected]>
collivier added a commit to collivier/functest that referenced this pull request May 22, 2019
Building functest-core container sometimes reaches travis-ci timeout
[0]. It decreases the compile time as proposed upstream [1].

[0] https://travis-ci.org/collivier/functest/jobs/531417730
[1] pyca/pynacl#303

Change-Id: Ie6c8e729f76c98c6f8c8e9ca01b2c998081f7ce8
Signed-off-by: Cédric Ollivier <[email protected]>
collivier added a commit to collivier/functest that referenced this pull request May 22, 2019
Building functest-core container sometimes reaches travis-ci timeout
[0]. It decreases the compile time as proposed upstream [1].

[0] https://travis-ci.org/collivier/functest/jobs/531417730
[1] pyca/pynacl#303

Change-Id: Ie6c8e729f76c98c6f8c8e9ca01b2c998081f7ce8
Signed-off-by: Cédric Ollivier <[email protected]>
b23prodtm added a commit to b23prodtm/acake2php that referenced this pull request Sep 11, 2019
b23prodtm added a commit to b23prodtm/acake2php that referenced this pull request Sep 11, 2019
b23prodtm added a commit to b23prodtm/acake2php that referenced this pull request Dec 10, 2019
DIND + DC

Composer php extension setup
fix Docker no space left on device

Docker tags

build-compose

Optimized docker built primary.dind images Docker-Compose install with pynacl : pyca/pynacl#303
b23prodtm added a commit to b23prodtm/acake2php that referenced this pull request Dec 10, 2019
Composer php extension setup
fix Docker no space left on device

Docker tags

build-compose

Optimized docker built primary.dind images Docker-Compose install with pynacl : pyca/pynacl#303
b23prodtm added a commit to b23prodtm/acake2php that referenced this pull request Feb 28, 2020
author b23production GNU <[email protected]> 1567975413 +0200
committer Tiana Rakoto Arimanana <[email protected]> 1582920287 +0100

DIND + DC

Composer php extension setup
fix Docker no space left on device

Docker tags

build-compose

Optimized docker built primary.dind images Docker-Compose install with pynacl : pyca/pynacl#303

php.ini error

Multi Arch Build

Enable disable balena arch dependent container block in YML

Workaround logger device /dev/log
balena-sound

No docker layer caching (not a free feature)

Export -f function not allowed in docker

Alias /dev/log on docker machine logger

balena docker hub

Discard sub project

Persistance

No prompt

CI build up

Docker Hub build Circle

dockerhub

WIP

Fix No package 'oniguruma' found

https://github.com/mlocati/docker-php-extension-installer

configure script

cross-compile added
aarch64 pushed to balena.io

Deploy scripts

Cross-build disable

apache2 php7 : New deployments
b23prodtm added a commit to b23prodtm/acake2php that referenced this pull request Feb 28, 2020
Composer php extension setup
fix Docker no space left on device

Docker tags

build-compose

Optimized docker built primary.dind images Docker-Compose install with pynacl : pyca/pynacl#303

php.ini error

Multi Arch Build

Enable disable balena arch dependent container block in YML

Workaround logger device /dev/log
balena-sound

No docker layer caching (not a free feature)

Export -f function not allowed in docker

Alias /dev/log on docker machine logger

balena docker hub

Discard sub project

Persistance

No prompt

CI build up

Docker Hub build Circle

dockerhub

WIP

Fix No package 'oniguruma' found

https://github.com/mlocati/docker-php-extension-installer

configure script

cross-compile added
aarch64 pushed to balena.io

Deploy scripts

Cross-build disable

apache2 php7 : New deployments
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

libsodium slow build time
2 participants