Skip to content

Update BUILDING.md#4255

Closed
kmatch98 wants to merge 1 commit into
adafruit:mainfrom
kmatch98:patch-1
Closed

Update BUILDING.md#4255
kmatch98 wants to merge 1 commit into
adafruit:mainfrom
kmatch98:patch-1

Conversation

@kmatch98
Copy link
Copy Markdown

Add --recursive to update command.

Add `--recursive` to update command.
Copy link
Copy Markdown
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

We actually have recommended not to do --recursive, because it drags in a lot of stuff to TinyUSB and some other submodules that isn't needed. I do not do --recursive myself and my builds work, so I think the original problem discussed in discord may have been due to something else.

@kmatch98
Copy link
Copy Markdown
Author

kmatch98 commented Feb 24, 2021

Ok, no problem. My builds were failing due to TinyUSB, so I had to make clean, then run the --recursive and then I could build properly. This was for the pyportal.

Feel free to ignore this if you think it was a different problem, but somehow this worked for me.

@dhalbert
Copy link
Copy Markdown
Collaborator

Discussed in discord # circuitpython, but for the record, this works on a fresh clone, without --recursive:

$ git clone https://github.com/adafruit/circuitpython
$ cd circuitpython/
$ git submodule update --init   # note no --recursive
$ cd ports/atmel-samd/
$ make BOARD=pyportal clean  # (unnecessary first time)
$ make BOARD=pyportal

@dhalbert dhalbert closed this Feb 24, 2021
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