Closed Bug 1078114 Opened 11 years ago Closed 11 years ago

[Stingray] Support tv_apps directory for CardUI related apps

Categories

(Firefox OS Graveyard :: Gaia::Build, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::Build
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2.1 S7 (24Oct)

People

(Reporter: rexboy, Assigned: chens)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(2 files)

Pull request
46 bytes, text/x-github-pull-request
rickychien
: review+
rickychien
: feedback+
Details | Review
Pull request
46 bytes, text/x-github-pull-request
rickychien
: review+
Details | Review
Per bug 1067781, we need to commit Card UI related apps inside tv_apps/ directory. So we need to modify build script to enable tv_apps/ directory for various situations, such as debugging under nightly, make install-gaia, etc.
Attached file Pull request —
Attachment #8500258 - Flags: feedback?(ricky060709)
Ricky, could you take a look and give some feedback? Once we have tv apps directory, this patch will enable debugging/testing tv apps from nightly.
I've no objections with create a tv_apps for TV device. It's fine to set r?=me. Thanks!
Attachment #8500258 - Flags: feedback?(ricky060709) → feedback+
Comment on attachment 8500258 [details] [review] Pull request Thanks Ricky :)
Attachment #8500258 - Flags: review?(ricky060709)
Comment on attachment 8500258 [details] [review] Pull request LGTM. r=@RickyChien
Attachment #8500258 - Flags: review?(ricky060709) → review+
master: https://github.com/mozilla-b2g/gaia/commit/39dc2d29aac89f36e8e26f9950cdb0b46f0f42b0
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee: nobody → shchen
See Also: → 1076699
Target Milestone: --- → 2.1 S7 (24Oct)
Whiteboard: [ft:conndevices]
When changing APP_TEST_LIST, you forgot to add $(APP) in the part for tv_apps, so as a result it tries to execute unit tests for tv_apps apps when we use APP=. For example: APP=sms make test-agent-test will launch tests for tv_apps. Can we please fix this?
Flags: needinfo?(shchen)
Sure, I can fix this. Thanks for reminding!
Flags: needinfo?(shchen)
Attached file Pull request —
Hi Ricky, Previous patch forgot to add $(APP) in tv_apps directory, and I've made this patch fixing this, would you review it? thanks!
Attachment #8517857 - Flags: review?(ricky060709)
Comment on attachment 8517857 [details] [review] Pull request LGTM. r+
Attachment #8517857 - Flags: review?(ricky060709) → review+
Thanks for the quick fix :)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: