New issue
Advanced search Search tips

Issue 625014 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

[remoting linux host] Debian package adds incorrect repo entry in APT sources list.

Project Member Reported by lambroslambrou@chromium.org, Jul 1 2016

Issue description

A typo in the file dropped into /etc/apt/sources.list.d (extra "}" on the "deb" line) is causing apt-get breakage.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9c6b6e1c9616f497d08be1413cbc0adf45a02224

commit 9c6b6e1c9616f497d08be1413cbc0adf45a02224
Author: lambroslambrou <lambroslambrou@chromium.org>
Date: Fri Jul 01 02:40:46 2016

[remoting linux host] Fix debian APT sources list entry.

This removes an extra '}' that accidentally got added in
http://crrev.com/20447ead8ba4b9bf3310da0d80a23a199aba1d60

BUG= 625014 

Review-Url: https://codereview.chromium.org/2111383002
Cr-Commit-Position: refs/heads/master@{#403395}

[modify] https://crrev.com/9c6b6e1c9616f497d08be1413cbc0adf45a02224/remoting/host/installer/linux/build-deb.sh

Project Member

Comment 2 by sheriffbot@chromium.org, Jul 1 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -M-54 -MovedFrom-53 Merge-Request-52 Merge-Request-53 M-52

Comment 4 by dimu@google.com, Jul 2 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)

Comment 5 by dimu@google.com, Jul 2 2016

Labels: -Merge-Request-53 Merge-Approved-53
Your change meets the bar and is auto-approved for M53 (branch: 2785)
Project Member

Comment 6 by sheriffbot@chromium.org, Jul 5 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 6 2016

Labels: -merge-approved-53 merge-merged-2785
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aa9c402837116d29ca1dc342eb24777f7ca4e580

commit aa9c402837116d29ca1dc342eb24777f7ca4e580
Author: Lambros Lambrou <lambroslambrou@chromium.org>
Date: Wed Jul 06 17:40:40 2016

[remoting linux host] Fix debian APT sources list entry.

This removes an extra '}' that accidentally got added in
http://crrev.com/20447ead8ba4b9bf3310da0d80a23a199aba1d60

BUG= 625014 

Review-Url: https://codereview.chromium.org/2111383002
Cr-Commit-Position: refs/heads/master@{#403395}
(cherry picked from commit 9c6b6e1c9616f497d08be1413cbc0adf45a02224)

Review URL: https://codereview.chromium.org/2126863003 .

Cr-Commit-Position: refs/branch-heads/2785@{#31}
Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382}

[modify] https://crrev.com/aa9c402837116d29ca1dc342eb24777f7ca4e580/remoting/host/installer/linux/build-deb.sh

Project Member

Comment 8 by bugdroid1@chromium.org, Jul 6 2016

Labels: -merge-approved-52 merge-merged-2743
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/31e9a56f817107e512dfaf661abad9f4492882d6

commit 31e9a56f817107e512dfaf661abad9f4492882d6
Author: Lambros Lambrou <lambroslambrou@chromium.org>
Date: Wed Jul 06 17:45:20 2016

[remoting linux host] Fix debian APT sources list entry.

This removes an extra '}' that accidentally got added in
http://crrev.com/20447ead8ba4b9bf3310da0d80a23a199aba1d60

BUG= 625014 

Review-Url: https://codereview.chromium.org/2111383002
Cr-Commit-Position: refs/heads/master@{#403395}
(cherry picked from commit 9c6b6e1c9616f497d08be1413cbc0adf45a02224)

Review URL: https://codereview.chromium.org/2119403003 .

Cr-Commit-Position: refs/branch-heads/2743@{#587}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/31e9a56f817107e512dfaf661abad9f4492882d6/remoting/host/installer/linux/build-deb.sh

Labels: -Hotlist-Merge-Approved
Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified fixed in 52.0.2743.48

Sign in to add a comment