Skip to content
This repository has been archived by the owner on Nov 30, 2019. It is now read-only.

Upgraded version of the build script and repo cleanup #2

Merged
merged 6 commits into from
Apr 20, 2013

Conversation

fmorgner
Copy link
Contributor

Hi

I'd like to thank you for the great work you've done with the build script. Your work motivated me to participate and "upgrade" the script a bit. I added improved parameter handling and renamed some stuff. I hope you like it and consider to pull my changes into your repo.

Best wishes,

Felix

@mangelajo
Copy link
Contributor

Wow, awesome work Felix!, please confirm that it builds on your side, I'll test here, and pull.

Miguel Angel Ajo
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Saturday, 20 de April de 2013 at 16:26, Felix Morgner wrote:

ok the error are fixed now. I'm still building, but everything should be fine now.


Reply to this email directly or view it on GitHub (#2 (comment)).

@fmorgner
Copy link
Contributor Author

I can confirm that the script builds fine. Thanks for your reply!

@mangelajo mangelajo merged commit d1e1011 into KiCad:master Apr 20, 2013
@mangelajo
Copy link
Contributor

Pulled! 👍 great work :)

@mangelajo
Copy link
Contributor

Thanks a lot for your work too, it make the script better :-)

@fmorgner
Copy link
Contributor Author

thanks, it's nice to hear this! I'm currently working on another "patch". should be ready in about 30-40 minutes. I'll open up another pull-request when I'm ready

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

Successfully merging this pull request may close these issues.

None yet

2 participants