Skip to content

Feature/better build #2

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
Open
kyle-neal wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Feature/better build #2

kyle-neal wants to merge 4 commits into from

Conversation

@kyle-neal
Copy link

@kyle-neal kyle-neal commented May 4, 2015

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

CorticalComputer commented May 4, 2015

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 notifications@github.com 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

kyle-neal commented May 4, 2015

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