ubuntu-system-settings

Merge lp://staging/~ted/ubuntu-system-settings/dev-mode-env into lp://staging/ubuntu-system-settings

  1. dev-mode-env
  2. Merge into trunk
Proposed by Ted Gould
Status: Needs review
Proposed branch: lp://staging/~ted/ubuntu-system-settings/dev-mode-env
Merge into: lp://staging/ubuntu-system-settings
Diff against target: 40 lines (+16/-0)
3 files modified
debian/developer-mode-settings.conf (+13/-0)
debian/rules (+2/-0)
debian/ubuntu-system-settings.install (+1/-0)
To merge this branch: bzr merge lp://staging/~ted/ubuntu-system-settings/dev-mode-env
Reviewer Review Type Date Requested Status
Sebastien Bacher (community) Disapprove
PS Jenkins bot continuous-integration Approve
Review via email: mp+247473@code.staging.launchpad.net

Commit message

Adding Upstart job to set debuging environment in developer mode

Description of the change

Small upstart job to set global environment variables in Qt and GLib for debug messages

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:1271
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-ci/1925/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/964
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-vivid-i386-ci/195
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/854
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/962
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/962/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/17401

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-ci/1925/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, but that job seems like something that should come with the session/touch init scripts, not with settings, it doesn't have much to do with the settings gui

review: Disapprove
Reply
Revision history for this message
Ted Gould (ted) wrote :

On Mon, 2015-01-26 at 09:15 +0000, Sebastien Bacher wrote:

> Thanks, but that job seems like something that should come with the
> session/touch init scripts, not with settings, it doesn't have much to
> do with the settings gui

I thought it should go with settings because that's where the setting
for how to configure developer mode is. So if we were to change how
developer mode is configured (I imagine we will at some point) that
could be handled all in the same package. The Upstart job just needs to
match the setting that the UI sets.

Reply

Unmerged revisions

1271. By Ted Gould

Attaching bug

1270. By Ted Gould

Make the directory as well

1269. By Ted Gould

Putting the config file in a package

1268. By Ted Gould

Adding an upstart job to use the developer mode setting for logging

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches