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

Rename folder "lib" to "libs" #3

Merged
merged 1 commit into from
Aug 19, 2014

Conversation

alvarogarcia7
Copy link
Contributor

So it gets exported to the .apk

The issue is this one: #2

So it gets exported to the .apk
@joeledstrom
Copy link
Owner

Thanks man. Sorry for not merging it earlier.

I have moved on to Android Studio, but I will download ADT tonight and test if this still even works.

(Also I noticed that I got a few local changes that were never committed to git. Quickly reviewing them it looks like it fixes some rather bad bug caused by not escaping XML escape chars. I will attempt to push those changes if I can get it working again.)

joeledstrom added a commit that referenced this pull request Aug 19, 2014
Rename folder "lib" to "libs"
@joeledstrom joeledstrom merged commit 9cc7b98 into joeledstrom:master Aug 19, 2014
@alvarogarcia7
Copy link
Contributor Author

Sorry for the long delay,

Any news on the ADT working?

thanks in advance,

On 20 August 2014 00:32, Joel Edström [email protected] wrote:

Merged #3 #3.


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

Alvaro García | Software Developer & Entrepreneur
@alvarobiz https://twitter.com/alvarobiz |
https://www.linkedin.com/in/alvarogarcia7

This message sent and any attachments (the "message") is intended solely
for the addressees and is confidential. If you receive this message in
error, please delete it and immediately notify the sender. Any use not in
accord with its purpose, any dissemination or disclosure, either whole or
partial, is prohibited except formal approval. The internet can not
guarantee the integrity of this message. The sender (and its subsidiaries)
shall (will) not therefore be liable for the message. Do not print this
message unless it is necessary, consider the environment.

@joeledstrom
Copy link
Owner

Yes. It did get it to compile and run after accepting your pull request. Thanks.

Also added a README and pushed my old local changes that I was working on back in 2011.
See the commit log.

@alvarogarcia7
Copy link
Contributor Author

Thanks for the effort. Deeply appreciated.

On 30 September 2014 20:52, Joel Edström [email protected] wrote:

Yes. It did get it to compile and run after accepting your pull request.
Thanks.

Also added a README and pushed my old local changes that I was working on
back in 2011. See the commit log
https://github.com/joeledstrom/SpreadsheetTest/commits/master


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

Alvaro García | Software Developer & Entrepreneur
@alvarobiz https://twitter.com/alvarobiz |
https://www.linkedin.com/in/alvarogarcia7

This message sent and any attachments (the "message") is intended solely
for the addressees and is confidential. If you receive this message in
error, please delete it and immediately notify the sender. Any use not in
accord with its purpose, any dissemination or disclosure, either whole or
partial, is prohibited except formal approval. The internet can not
guarantee the integrity of this message. The sender (and its subsidiaries)
shall (will) not therefore be liable for the message. Do not print this
message unless it is necessary, consider the environment.

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.

2 participants