Opened on 11/07/2017 at 06:34:44 AM
Closed on 05/16/2018 at 05:13:59 PM
#6000 closed change (fixed)
Rename "libadblockplus-android"
Reported by: | asmirnov | Assignee: | |
---|---|---|---|
Priority: | P4 | Milestone: | |
Module: | Libadblockplus-Android | Keywords: | |
Cc: | fhd, sergz, diegocarloslima, jwangenheim, rjeschke | Blocked By: | #6300 |
Blocking: | #6205 | Platform: | Android |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://codereview.adblockplus.org/29678581/ |
Description
Background
For historical reason and to be consistent with other eyeo products libadblockplus-android is named the way it is at the moment. However this seems to be confusing for Android developers (according to Felix at android dev meeting) and should be improved.
What to change
Rename libadblockplus-android to something more common to Android developers.
Attachments (0)
Change History (17)
comment:1 Changed on 11/07/2017 at 06:42:35 AM by asmirnov
comment:2 Changed on 11/07/2017 at 06:43:05 AM by asmirnov
Android guys, what's your suggestion? I's name it "Adblock Plus Android SDK"
comment:3 Changed on 11/07/2017 at 06:44:33 AM by asmirnov
- Cc rjeschke added
comment:4 follow-up: ↓ 5 Changed on 11/07/2017 at 10:26:40 AM by jwangenheim
I think 'Adblock Plus Android SDK' would be a good choice . The only alternative I could imagine would be something like "Adblock WebView" but
- We offer more than only a WebView that blocks ads and probably we want to extend the functionality in the future
- It makes more sense to have the product name reflected in the name of the lib.
comment:5 in reply to: ↑ 4 Changed on 11/07/2017 at 10:31:36 AM by asmirnov
Replying to jwangenheim:
I think 'Adblock Plus Android SDK' would be a good choice . The only alternative I could imagine would be something like "Adblock WebView" but
- We offer more than only a WebView that blocks ads and probably we want to extend the functionality in the future
BTW AdblockWebView is not the only module provided and the users can use 'libadblockplus-android' submodule without 'libadblockplus-android-webview' used.
- It makes more sense to have the product name reflected in the name of the lib.
comment:6 Changed on 11/07/2017 at 10:58:37 AM by jwangenheim
Yeah, that's exactly what I meant by 1. :) I just could image Adblock-Webview as a name for the lib, becuse it's probably the most striking and easy to implement feature.
comment:7 Changed on 11/07/2017 at 11:26:13 AM by sergz
I like Adblock Plus Android SDK.
In order to handle WebView I would like to firstly know what will be changed when the product is renamed. In particular if the everything continues to live in the same repo libadblockplus-android then we can have proper names for each component, one has to just figure out how to properly work with them, namely, build, distribute and convey, etc. E.g. take a look at spotify, they have SDK consisting from several libraries and it's shortly explain in The SDK's structure section.
comment:8 Changed on 11/07/2017 at 02:20:05 PM by fhd
I think we can leave out the _Plus_, we did the same for Adblock Browser and AdblockWebView. As in Adblock Android SDK.
comment:9 Changed on 11/09/2017 at 01:38:39 PM by diegocarloslima
I would suggest the full name for README /docs to be Adblock SDK for Android and the contracted version for ids / repo name to be Adblock-SDK-Android
comment:10 Changed on 12/12/2017 at 01:53:21 PM by jwangenheim
Hey guys,
should we make a final decision here? I like 'Adblock Android SDK' as a name for the whole repository. But I think it would also make sense to distribute libadblockplus-android, libadblockplus-android-settings and libadblockplus-android-webview as single maven dependencies, to make adblocking in Android apps available as fast and easy as possible.
comment:11 Changed on 12/12/2017 at 02:20:38 PM by asmirnov
it would also make sense to distribute libadblockplus-android, libadblockplus-android-settings and libadblockplus-android-webview as single maven dependencies
I don't think it's good idea, i see no point in making it single module at all. Using of 'libadblockplus-android' does not mean using 'libadblockplus-android-webview', similar to settings - they are independent of WebView (see https://github.com/adblockplus/libadblockplus-android/blob/master/libadblockplus-android-settings/pom.xml).
What about module dependencies i prefer leave everything as-is, but just rename the modules.
comment:12 Changed on 12/18/2017 at 02:12:09 PM by diegocarloslima
- Blocking 6205 added
comment:13 Changed on 01/24/2018 at 06:08:01 AM by asmirnov
- Blocked By 6300 added
comment:14 Changed on 01/24/2018 at 07:08:55 AM by asmirnov
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:15 Changed on 05/15/2018 at 09:40:49 PM by diegocarloslima
- Review URL(s) modified (diff)
comment:16 Changed on 05/16/2018 at 04:33:42 PM by abpbot
A commit referencing this issue has landed:
Issue 6000 - Rename "libadblockplus-android"
comment:17 Changed on 05/16/2018 at 05:13:59 PM by diegocarloslima
- Resolution set to fixed
- Status changed from reviewing to closed
Usually it's named as "Product X Android SDK", see: