Skip to content

remove autogenerated eagle.app.v6.common.ld and git-ignore it #4618

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.

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

Merged
merged 1 commit into from
Apr 22, 2018

Conversation

d-a-v
Copy link
Collaborator

@d-a-v d-a-v commented Apr 7, 2018

No description provided.

@d-a-v d-a-v requested a review from igrr April 7, 2018 20:59
@igrr
Copy link
Member

igrr commented Apr 7, 2018

Let me guess: LD script is not auto-generated when building with platformio?

@d-a-v
Copy link
Collaborator Author

d-a-v commented Apr 7, 2018

well... yes.
@ivankravets could you help with this ?
ref: #4582

@igrr
Copy link
Member

igrr commented Apr 10, 2018

Paging @ivankravets, please help us with platformio build script!

We need to run this line of platform.txt: https://github.com/esp8266/Arduino/pull/4582/files#diff-d638a47d1ca0e2a5585fb34c7d7fb0f9R82 as part of platformio build process.

@ivankravets
Copy link
Collaborator

@igrr sorry for the delay, I'm on a business trip to the USA now. I'll update build script soon to generate common LD script on the fly.

@ivankravets
Copy link
Collaborator

@d-a-v could you resolve conflicts or create new PR? All should work now.

@d-a-v
Copy link
Collaborator Author

d-a-v commented Apr 22, 2018

@ivankravets thanks. I puffed this PR.

(push -f ed)

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

Successfully merging this pull request may close these issues.

3 participants