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

Feature/better build #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Feature/better build #2

wants to merge 4 commits into from

Conversation

kyle-neal
Copy link

This is just a better system for building the code.

To build:
make

Clean (removes all */ebin/.beam files):
make clean

Run (actually loads the modules as well!):
make run_Ch_##

@CorticalComputer
Copy link
Owner

Hello Kyle,

This looks great, I want to add it, but the files and their setup, because
I reference them directly from the book, if I change them, the book will no
longer match up, and people will get confused when they follow the book
since some of the code will be structured differently.

I can set this up for the current system, DXNN V2 on GitHub, and the
current version on my machine, DXNN V3.

Best regards,
-Gene

On Mon, May 4, 2015 at 4:31 PM, Kyle Neal [email protected] wrote:

kyle-neal wants to merge 4 commits into CorticalComputer:master from
kyle-neal:feature/BetterBuild:

This is just a better system for building the code.

To build:
make

Clean (removes all */ebin/.beam files):
make clean

Run (actually loads the modules as well!):

make run_Ch_##

You can view, comment on, or merge this pull request online at:

#2
Commit Summary

  • (chore) #noticket Add an ignore to idea files.
  • (chore) Moved all files to better structure for build.
  • (chore) Ignore the ebin directories.
  • (feat) Created a makefile that will build/run/clean all code!

File Changes

Patch Links:

https://github.com/CorticalComputer/Book_NeuroevolutionThroughErlang/pull/2.patch

https://github.com/CorticalComputer/Book_NeuroevolutionThroughErlang/pull/2.diff

—
Reply to this email directly or view it on GitHub
#2
.

@kyle-neal
Copy link
Author

Ah I see, I didn't think about that. I assume because I changed the structure to be Ch_#/src/... If you're interested in implementing this in the current repo and leave the structure, I could just change the layout for the code to still be in Ch_#/, but I would still have to add in Ch_#/ebin (only done on local machines) since you do have files with the same name. Should be a very simple change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kyle-neal @CorticalComputer