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

Fixed boards.txt path in Cosa.mk #502

Closed
wants to merge 1 commit into from

Conversation

Mogball
Copy link
Contributor

@Mogball Mogball commented Dec 17, 2017

I'm running Cosa withgcc-avr on MacOS and Ubuntu 16.04.

BOARDS_TXT = $(COSA_DIR)/build/boards.txt

But the actual path is $COSA_DIR/boards.txt. This results in no output from running cosa boards and an error when trying to build using cosa [BOARD] (missing device or architecture after '-mmcu=')

#470 seems to be related?

@Mogball
Copy link
Contributor Author

Mogball commented Dec 17, 2017

After looking at some other recent PRs, seems I might've been presumptuous on this one! Will reopen if I figure something out.

@Mogball Mogball closed this Dec 17, 2017
@mikaelpatel
Copy link
Owner

@Mogball

Thanks for your interest in this project. The idea behind the boards.txt is that it is generated from different board contributions. Please see https://github.com/mikaelpatel/Cosa/blob/master/build/Arduino-Makefile/Arduino.mk#L535

@Mogball
Copy link
Contributor Author

Mogball commented Dec 17, 2017

Yeah I did notice that. But it wouldn't work if Arduino IDE is not installed right? If I understand correctly that seems to be what #470 is trying to address.

@mikaelpatel
Copy link
Owner

@Mogball There is no support for the Arduino IDE to generate the merged boards.txt files. This has to be done manually (unfortunately).

@Mogball
Copy link
Contributor Author

Mogball commented Dec 17, 2017

Alright, that's no problem. I'm looking into developing a build system based around cosa anyway.

Thanks for your responses!

@Mogball Mogball deleted the jeff/boards_dir branch December 17, 2017 21:55
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