Closed
Bug 1084521
Opened 10 years ago
Closed 10 years ago
did you mean to use @+id instead of @+android:id?
Categories
(Android Background Services Graveyard :: Firefox Accounts, defect)
Tracking
(firefox34 fixed, firefox35 fixed, firefox36 fixed)
RESOLVED
FIXED
Firefox 36
People
(Reporter: rnewman, Assigned: rnewman)
References
Details
Attachments
(1 file)
3.48 KB,
patch
|
lucasr
:
review+
lsblakk
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:37: note: did you mean to use @+id instead of @+android:id? 21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:48: note: did you mean to use @+id instead of @+android:id? 21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:90: note: did you mean to use @+id instead of @+android:id? 21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:101: note: did you mean to use @+id instead of @+android:id?
Assignee | ||
Comment 1•10 years ago
|
||
Probably causes this, too: 21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:37: error: creating resource for external package android: id/account_server_title. 21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:37: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/account_server_title'). 21:28.09 21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:48: error: creating resource for external package android: id/account_server_summary. 21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:48: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/account_server_summary'). 21:28.09 21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:90: error: creating resource for external package android: id/sync_server_title. 21:28.10 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:90: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/sync_server_title'). 21:28.10 21:28.10 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:101: error: creating resource for external package android: id/sync_server_summary. 21:28.10 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:101: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/sync_server_summary').
Assignee | ||
Comment 2•10 years ago
|
||
This looks like an over-zealous s//. Uplift needed.
Attachment #8507228 -
Flags: review?(nalexander)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
https://github.com/mozilla-services/android-sync/pull/503
Comment 4•10 years ago
|
||
Comment on attachment 8507228 [details] [diff] [review] Use +id not +android:id. v1 Review of attachment 8507228 [details] [diff] [review]: ----------------------------------------------------------------- Yep.
Attachment #8507228 -
Flags: review?(nalexander) → review+
Assignee | ||
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/eb04720f2290
Assignee | ||
Comment 6•10 years ago
|
||
https://github.com/mozilla-services/android-sync/commit/0e4ebcf489f1bbd85eea038a0218006c5c76421a
https://hg.mozilla.org/mozilla-central/rev/eb04720f2290
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Comment 8•10 years ago
|
||
This patch applies cleanly to (at least) the beta branch (firefox-34) and fixes the same issue that occurs when building against that branch as well. I haven't tested if the patch applies cleanly to aurora (firefox-35), but I would assume that it does.
Comment 9•10 years ago
|
||
Comment on attachment 8507228 [details] [diff] [review] Use +id not +android:id. v1 Approval Request Comment [Feature/regressing bug #]: no idea. [User impact if declined]: none; this is build only. [Describe test coverage new/current, TBPL]: none needed, this is build only. [Risks and why]: none. [String/UUID change made/needed]: none.
Attachment #8507228 -
Flags: approval-mozilla-beta?
Attachment #8507228 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
Attachment #8507228 -
Flags: approval-mozilla-beta?
Attachment #8507228 -
Flags: approval-mozilla-beta+
Attachment #8507228 -
Flags: approval-mozilla-aurora?
Attachment #8507228 -
Flags: approval-mozilla-aurora+
Comment 10•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/fba3e828d330 https://hg.mozilla.org/releases/mozilla-beta/rev/81b50459db25
You need to log in
before you can comment on or make changes to this bug.
Description
•