Closed Bug 1121392 Opened 5 years ago Closed 5 years ago

Add bluetooth daemon to lollipop manifest

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

x86_64
Linux
defect
Not set

Tracking

(feature-b2g:2.2+, b2g-v2.2 fixed)

RESOLVED FIXED
2.2 S4 (23jan)
feature-b2g 2.2+
Tracking Status
b2g-v2.2 --- fixed

People

(Reporter: shawnjohnjr, Assigned: shawnjohnjr)

References

Details

Attachments

(2 files)

Add bluetooth daemon to lollipop manifest
This build config to fetch daemon code is required for 2.2.
Assignee: nobody → shuang
feature-b2g: --- → 2.2+
Target Milestone: --- → 2.2 S4 (23jan)
FYI: The blueprint for this patch is in bug 1104620. Bluetoothd depends on libfdio.
Comment on attachment 8548819 [details] [review]
Bug 1121392 - Add bluetooth daemon to lollipop manifest

Should work, but maybe sorting alphabetically by the project name increases the readability of the file. Does the change get ported to v2.2 by Sheriffs?

Bruce, did you need any other repository for building on L? Such as 'system/bluetooth'?

Also, asking :mwu for an official review.
Attachment #8548819 - Flags: review?(tzimmermann)
Attachment #8548819 - Flags: review?(mwu)
Attachment #8548819 - Flags: feedback?(brsun)
(In reply to Thomas Zimmermann [:tzimmermann] [:tdz] from comment #4)
> Comment on attachment 8548819 [details] [review]
> Bug 1121392 - Add bluetooth daemon to lollipop manifest
> 
> Bruce, did you need any other repository for building on L? Such as
> 'system/bluetooth'?

No. I can build 'system/libfdio', 'system/bluetoothd', and Gecko on L without 'system/bluetooth'.
Attachment #8548819 - Flags: feedback?(brsun) → feedback+
(In reply to Thomas Zimmermann [:tzimmermann] [:tdz] from comment #4)
> Comment on attachment 8548819 [details] [review]
> Bug 1121392 - Add bluetooth daemon to lollipop manifest
> 
> Should work, but maybe sorting alphabetically by the project name increases
> the readability of the file.
Sure, I will change the order.
> Does the change get ported to v2.2 by Sheriffs?
It should get ported to v2.2 by sheriffs.
I guess it can be directly in v2.2, like https://bugzilla.mozilla.org/show_bug.cgi?id=1106858#c27 

For system/bluetooth, Lollipop codebase already removed this git repository.
Blocks: gonk-L-BT
Attachment #8548819 - Flags: review?(mwu) → review?(kli)
The patch is similar with base-kk.xml, it seems mwu is busy, I redirect review to Kai-Zhen for this simple manifest patch (2.2 feature set).
Shawn, I think the repo name is not correct and I left message on PR. Can you update PR with correct repo name?
Flags: needinfo?(shuang)
(In reply to Kai-Zhen Li [:seinlin] from comment #8)
> Shawn, I think the repo name is not correct and I left message on PR. Can
> you update PR with correct repo name?

I updated the patch during sorting alphabetically, I was wrong.
I've already update the patch again, thanks.
Flags: needinfo?(shuang)
Comment on attachment 8548819 [details] [review]
Bug 1121392 - Add bluetooth daemon to lollipop manifest

Looks good to me.
Attachment #8548819 - Flags: review?(kli) → review+
https://github.com/mozilla-b2g/b2g-manifest/commit/2f921fea805479a4ffdca44e1ab031a69c6cdcc5

This is a 2.2+ bug, Could you also send a PR to v2.2 branch?
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(shuang)
Resolution: --- → FIXED
Only nexus-5-l.xml is available but no base-l.xml in b2g-manifest v2.2 branch. So for v2.2, nexus-5-l.xml will be changed.
Flags: needinfo?(shuang)
You need to log in before you can comment on or make changes to this bug.