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.

Sign up for GitHub

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

Jump to bottom

make rebuilds programs #2682

Closed
gilles-peskine-arm opened this issue Jun 7, 2019 · 0 comments · Fixed by #2697
Closed

make rebuilds programs #2682

gilles-peskine-arm opened this issue Jun 7, 2019 · 0 comments · Fixed by #2697
Labels
bug component-platform Portability layer and build scripts

Comments

@gilles-peskine-arm
Copy link
Contributor

Description

  • Type: Bug
  • Priority: Minor

Bug

Since some submodule-related change, make rebuilds programs even when they don't need to be rebuilt.

Expected behavior

After a successful build, running make again does nothing.

Actual behavior

After a successful build, running make again rebuilds everything under programs.

Steps to reproduce

make
make
@gilles-peskine-arm gilles-peskine-arm added bug component-platform Portability layer and build scripts labels Jun 7, 2019
gilles-peskine-arm added a commit to gilles-peskine-arm/mbedtls that referenced this issue Jun 7, 2019
@gilles-peskine-arm
Don't systematically rebuild programs
1830b71 
Fix the dependency on libmbedcrypto.a, which is now located under
crypto.

Fix Mbed-TLS#2682
gilles-peskine-arm added a commit to gilles-peskine-arm/mbedtls that referenced this issue Jun 7, 2019
@gilles-peskine-arm
Don't systematically rebuild programs
9051036 
Fix the dependency on libmbedcrypto.a, which is now located under
crypto.

Fix Mbed-TLS#2682
gilles-peskine-arm added a commit to gilles-peskine-arm/mbedtls that referenced this issue Jun 7, 2019
@gilles-peskine-arm
Don't systematically rebuild programs
95f5cbc 
Fix the dependency on libmbedcrypto.a, which is now located under
crypto.

Fix Mbed-TLS#2682
This was referenced Jun 7, 2019
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug component-platform Portability layer and build scripts
Projects
None yet
Milestone
No milestone
1 participant
@gilles-peskine-arm