Skip to content
This repository has been archived by the owner on Mar 9, 2023. It is now read-only.

Multi-project support in same or separate repo? #21

Open
jrrk2 opened this issue Feb 21, 2022 · 6 comments
Open

Multi-project support in same or separate repo? #21

jrrk2 opened this issue Feb 21, 2022 · 6 comments
Labels
question Further information is requested

Comments

@jrrk2
Copy link
Contributor

jrrk2 commented Feb 21, 2022

xc7k325-blinky-nextpnr only supports one project at the moment. Opened a new branch:
project_select in git@github.com:jrrk2/xc7k325t-blinky-nextpnr.git to allow picosoc to be built using the same Makefile. This invalidates the name of the project, so this is just a placeholder to discuss what should be done to make it scalable.

This new project builds fine and flashes on an Apple OSX M1 machine.

@unbtorsten
Copy link
Contributor

I suggest waiting for @rwhitby to finish refactoring the make procedure, cf. #18. Either way, it may be reasonable to create a separate repo for the picosoc process (and other designs in the future)?

@unbtorsten unbtorsten added the question Further information is requested label Feb 23, 2022
@unbtorsten unbtorsten changed the title Only one project supported Multi-project support in same or separate repo? Feb 23, 2022
@rwhitby
Copy link
Contributor

rwhitby commented Feb 24, 2022

rwhitby#1 has files distinguished by ${PROJECT_NAME} and ${BOARD}, so multiple projects in the same directory should be possible. Whether or not that is a good idea is a separate question :-)

@unbtorsten
Copy link
Contributor

Looks good. :) At minimum, this makes the Makefile flexible enough to be used for other projects.
Can you change its first line from

PROJECT_NAME = blinky

to

PROJECT_NAME ?= blinky

That way, the project name can be set when calling make.

@jrrk2
Copy link
Contributor Author

jrrk2 commented Feb 24, 2022

OK done, but I think command line variable overrides take priority over =, = takes priority over the environment, and environment takes priority over ?=

@rwhitby
Copy link
Contributor

rwhitby commented Feb 25, 2022

Looks good. :) At minimum, this makes the Makefile flexible enough to be used for other projects. Can you change its first line from

PROJECT_NAME = blinky

to

PROJECT_NAME ?= blinky

That way, the project name can be set when calling make.

Done

@rwhitby
Copy link
Contributor

rwhitby commented Nov 26, 2022

See #33 for how this can be done in the same repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants