Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(134)

Issue 6674010: Remove old Xservers. (Closed)

Created:
9 years, 9 months ago by marcheu
Modified:
9 years, 7 months ago
Reviewers:
kliegs, anush
CC:
chromium-os-reviews_chromium.org, msb+crosoverlay_chromium.org, Daniel Erat, adlr+crosoverlay_chromium.org, anush
Visibility:
Public.

Description

Remove old Xservers. We shouldn't need those any more. Change-Id: I94f52c9dafb29db4e5fecb596f4365969a3daf99 BUG=None TEST=Check that chrome OS builds, run it, look at /var/log/Xorg.0.log and check that xserver 1.9 or better is used. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=59ba3a3

Patch Set 1 #

Patch Set 2 : Remove old xserver ebuilds. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1119 lines) Patch
M x11-base/xorg-server/xorg-server-1.6.5-r200.ebuild View 1 1 chunk +0 lines, -266 lines 0 comments Download
D x11-base/xorg-server/xorg-server-1.6.5-r203.ebuild View 1 1 chunk +0 lines, -1 line 0 comments Download
M x11-base/xorg-server/xorg-server-1.7.6.ebuild View 1 1 chunk +0 lines, -299 lines 0 comments Download
D x11-base/xorg-server/xorg-server-1.7.6-r16.ebuild View 1 1 chunk +0 lines, -1 line 0 comments Download
D x11-base/xorg-server/xorg-server-1.8.0.ebuild View 1 1 chunk +0 lines, -276 lines 0 comments Download
D x11-base/xorg-server/xorg-server-1.8.0-r3.ebuild View 1 1 chunk +0 lines, -1 line 0 comments Download
D x11-base/xorg-server/xorg-server-1.8.2.ebuild View 1 1 chunk +0 lines, -274 lines 0 comments Download
D x11-base/xorg-server/xorg-server-1.8.2-r1.ebuild View 1 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
marcheu
9 years, 9 months ago (2011-03-11 01:44:35 UTC) #1
anush
Can we delete them? They are available in git if we want them back. On ...
9 years, 9 months ago (2011-03-11 01:46:44 UTC) #2
marcheu
On 2011/03/11 01:46:44, anush wrote: > Can we delete them? They are available in git ...
9 years, 9 months ago (2011-03-11 02:27:20 UTC) #3
marcheu
On 2011/03/11 02:27:20, marcheu wrote: > On 2011/03/11 01:46:44, anush wrote: > > Can we ...
9 years, 9 months ago (2011-03-11 02:34:38 UTC) #4
anush
Rietveld is showing it as modified for some reason. LGTM as long as all the ...
9 years, 9 months ago (2011-03-11 06:32:18 UTC) #5
kliegs
9 years, 9 months ago (2011-03-11 15:46:24 UTC) #6
LGTM as long as you update the BUG= and TEST= fields.

Will be good to get rid of them all from disk to catch any builds that might
accidentally be using 1.9.3.  

On 2011/03/11 06:32:18, anush wrote:
> Rietveld is showing it as modified for some reason. LGTM as long as all the
> files on CL are actually deletes.
> 
> On Thu, Mar 10, 2011 at 6:34 PM, <mailto:marcheu@chromium.org> wrote:
> 
> > On 2011/03/11 02:27:20, marcheu wrote:
> >
> >> On 2011/03/11 01:46:44, anush wrote:
> >> > Can we delete them? They are available in git if we want them back.
> >>
> >
> >  You're right, lets do that.
> >>
> >
> > Ok, not I'm deleting them. Please take another look!
> >
> >
> > http://codereview.chromium.org/6674010/
> >

Powered by Google App Engine
This is Rietveld 408576698