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

ActionBarSherlock is always required dependency #5

Open
futtetennista opened this issue Nov 29, 2012 · 7 comments
Open

ActionBarSherlock is always required dependency #5

futtetennista opened this issue Nov 29, 2012 · 7 comments
Labels
Milestone

Comments

@futtetennista
Copy link

Hi Joel,

Thanks very much for this library, it's extremely useful and it works just fine. One issue I had though is that it always requires ActionBarSherlock as a dependency and I don't think one should be forced to use it - even if one could argue that is not such a wise choice. I worked on removing ABS and adding support to it in a separate project, taking the same approach Jake Wharton took with the ABS Map plugin. You can check these two links out and tell me what you think about it, if you think it's fine I'll open a pull request: https://github.com/stefanodacchille/UnifiedPreference/tree/plugin-actionbarsherlock, and https://github.com/stefanodacchille/UnifiedPreference-ActionBarSherlockPlugin .

@saik0
Copy link
Owner

saik0 commented Nov 29, 2012

Nice work. I will cherry-pick from your repo. I do not want to change attr names.

@futtetennista
Copy link
Author

Thanks. Not sure it'll work without changing those attribute name though, I remember trying to do that and failing. But maybe I was just missing something.

@saik0
Copy link
Owner

saik0 commented Dec 1, 2012

Ah, I see what's happening with those duplicate attrs now. Very frustrating.

@saik0 saik0 reopened this Dec 1, 2012
@saik0
Copy link
Owner

saik0 commented Dec 1, 2012

Thanks. Not sure it'll work without changing those attribute name though, I remember trying to do that and failing. But maybe I was just missing something.

Library projects don't have their own attribute namespace, they get merged into your project. Before we were using the title and icon attrs defined in ABS, note how the xml nodes had no format attribute. Hopefully the "new build system" will have proper libraries, but I suspect it wont be released until after ABS and this project are not needed.

@futtetennista
Copy link
Author

Of course, you're right. Then unfortunately those must change. It sucks
that is a breaking change but I think it's still a cleaner approach.
On 1 Dec 2012 07:31, "Joel Pedraza" [email protected] wrote:

Thanks. Not sure it'll work without changing those attribute name though,
I remember trying to do that and failing. But maybe I was just missing
something.

Library projects don't have their own attribute namespace, they get merged
into your project. Before we were using the title and icon attrs defined in
ABS, note how the xml nodes had no format attribute. Hopefully the "new
build system" will have proper libraries, but I suspect it wont be released
until after ABS and this project are not needed.


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-10914175.

@saik0
Copy link
Owner

saik0 commented Dec 1, 2012

Did you manage to get the demo to build with maven? I get this error:

UnifiedPreference-ActionBarSherlockPlugin/demo/AndroidManifest.xml:28: error: Error: No resource found that matches the given name (at 'theme' with value style/Theme.Sherlock').

@futtetennista
Copy link
Author

Uhm, no I can build the project without errors. I fixed a crash at Runtime
when using Maven to build the app though: I forgot to set the correct
source directory in plugin and demo POM files.

On 1 December 2012 22:38, Joel Pedraza [email protected] wrote:

Did you manage to get the demo to build with maven? I get this error:

UnifiedPreference-ActionBarSherlockPlugin/demo/AndroidManifest.xml:28:
error: Error: No resource found that matches the given name (at 'theme'
with value style/Theme.Sherlock').


Reply to this email directly or view it on GitHubhttps://github.com//issues/5#issuecomment-10922498.

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

No branches or pull requests

2 participants