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.

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

no app-*.o files were found in my target/debug... folder. I am on ru… #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

U007D
Copy link
Contributor

@U007D U007D commented Jan 30, 2019

…stc 1.32.0 on Ubuntu 18.04; updated instructions to remove .o; note output was also not as shown--corrected as well.

If this PR is accepted we should update the app.nm #include, remove the inline dump and 'Note: as opposed to:' text.

…tc 1.32.0 on Ubuntu 18.04; updated instructions to remove .o; note output was also not as shown--corrected as well.
@therealprof
Copy link
Contributor

Hm, not sure this is the best option as this will also weed unnecessarily through all other files in that directory.

@U007D
Copy link
Contributor Author

U007D commented Feb 1, 2019

Current implementation shows all app-*.o, which do not exist (at least on my machine).

Suggestions?

@Disasm Disasm added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-resources labels Feb 26, 2019
@jamesmunns
Copy link
Member

@U007D was this specifically on a mac? Do you happen to still have the same configuration? I'd be interested to know what was there.

@jamesmunns
Copy link
Member

Note: will probably need to be rebased on #66

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-resources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants