phablet-test-run -c option fails because it tries to adb push to a directory owned by root

Bug #1411697 reported by Brendan Donegan
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
phablet-tools (Ubuntu)
In Progress
High
Brendan Donegan

Bug Description

phablet-test-run creates a temp directory called (oddly) /tmp/phablet-run-test/ to push .deb files to when specified by the -c option. Unfortunately it uses exec_with_adb to do this which creates the directory as root. Then it uses adb push to copy the file which files with a permissions error. Creating the directory and copying the file don't require root so this should be done with exec_with_adb_user instead.

ProblemType: Bug
DistroRelease: Ubuntu 14.10
Package: phablet-tools 1.1+15.04.20141107.1-0ubuntu1 [modified: usr/bin/phablet-test-run] [origin: unknown]
Uname: Linux 3.13.0-031300rc2-generic x86_64
ApportVersion: 2.14.7-0ubuntu8
Architecture: amd64
CurrentDesktop: Unity
Date: Fri Jan 16 14:39:33 2015
InstallationDate: Installed on 2013-11-30 (412 days ago)
InstallationMedia: Ubuntu 13.10 "Saucy Salamander" - Release amd64 (20131016.1)
PackageArchitecture: all
SourcePackage: phablet-tools
SystemImageInfo:
 current build number: 0
 device name:
 channel: daily
 last update: Unknown
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in phablet-tools (Ubuntu):
status: New → Confirmed
Changed in phablet-tools (Ubuntu):
assignee: nobody → Brendan Donegan (brendan-donegan)
status: Confirmed → In Progress
Changed in phablet-tools (Ubuntu):
importance: Undecided → High
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.