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

Makefile: Error out on missing submodules #710

Merged
merged 1 commit into from
Mar 2, 2021

Conversation

jonasdn
Copy link
Contributor

@jonasdn jonasdn commented Mar 2, 2021

If you, like me, fail to read the README.md properly you will get a bit of a cryptic error when trying to build.

   $ make
   CLEAN_VERSION
   make[1]: *** No rule to make target 'list.o', needed by 'cf2.elf'.  Stop.
   make: *** [Makefile:380: build] Error 2

This does not make it super clear that it is the missing submodules that is at fault.
This PR adds a check for the vendor submodules and will produce an error if missing, like so:

   $ make
   Makefile:383: ***
    The submodules does not seem to be present, consider fetching them by:
    $ git submodule init
    $ git submodule update
    .  Stop.

@jonasdn jonasdn force-pushed the jonasdn/error-submodules branch 3 times, most recently from 11bb4a7 to 76996f1 Compare March 2, 2021 05:28
@jonasdn jonasdn changed the title Build: Error out on missing submodules Makefile: Error out on missing submodules Mar 2, 2021
@jonasdn jonasdn force-pushed the jonasdn/error-submodules branch from 76996f1 to b78cbed Compare March 2, 2021 05:31
 If you, like me, fail to read the README.md properly you will get a bit
 of a cryptic error when trying to build.

   $ make
   CLEAN_VERSION
   make[1]: *** No rule to make target 'list.o', needed by 'cf2.elf'.  Stop.
   make: *** [Makefile:380: build] Error 2

 This does not make it super clear that it is the missing submodules that
 is at fault.

 This commit adds a check for the vendor submodules and will produce an
 error if missing, like so:

   $ make
   Makefile:383: ***
    The submodules does not seem to be present, consider fetching them by:
    $ git submodule init
    $ git submodule update
    .  Stop.
@jonasdn jonasdn force-pushed the jonasdn/error-submodules branch from b78cbed to c996bee Compare March 2, 2021 05:52
@jonasdn
Copy link
Contributor Author

jonasdn commented Mar 2, 2021

Another way to tackle this would be to just fetch the submodules if they are not there on make. Since a call to makeimplies that a user wants to build the firmware. And they can't without the submodules.

@jonasdn jonasdn merged commit 007e327 into bitcraze:master Mar 2, 2021
@jonasdn jonasdn added this to the next-release milestone Mar 2, 2021
@jonasdn jonasdn deleted the jonasdn/error-submodules branch March 2, 2021 10:00
cafeciaojoe pushed a commit to cafeciaojoe/crazyflie-firmware that referenced this pull request Sep 27, 2024
…r_config

inputconfigdialogue: Update buttonrole enum to pyqt6.
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.

1 participant