Skip to content
This repository has been archived by the owner on Jun 2, 2021. It is now read-only.

mining.ini within installed package #230

Closed
ayr-ton opened this issue Jan 27, 2015 · 4 comments · Fixed by #231
Closed

mining.ini within installed package #230

ayr-ton opened this issue Jan 27, 2015 · 4 comments · Fixed by #231

Comments

@ayr-ton
Copy link
Member

ayr-ton commented Jan 27, 2015

When build the mining for production, it doesn't copy the mining.ini file within the package.

Suggestions:

  • Under setup(), include a static session "data_files" and add "mining.ini" to this.
  • A different default place for the conf files for production installs, that doesn't depend from PROJECT_PATH variable
  • Other?
@avelino
Copy link
Collaborator

avelino commented Jan 27, 2015

Very nice, It had left the process simpler.

You send PR?

@ayr-ton
Copy link
Member Author

ayr-ton commented Jan 27, 2015

Yes, sure.
Which one of the suggestions do you guys prefer?
Can I choose?

@avelino
Copy link
Collaborator

avelino commented Jan 27, 2015

Stay so to implement the best solution!

@ayr-ton ayr-ton mentioned this issue Jan 27, 2015
Merged
@ayr-ton
Copy link
Member Author

ayr-ton commented Jan 27, 2015

@avelino I choose the first suggestion. In the future we can discuss a better place for the conf, but it solves for now.

@ayr-ton ayr-ton mentioned this issue Feb 3, 2015
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add mining.ini to egg and update README.rst to fit ayr-ton/mining
2 participants
@avelino @ayr-ton