1464834 - Remove dead Servo code.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Description

[Emilio Cobos Álvarez (:emilio)]

It's not part of the build.

Comment 1

[Phabricator Automation]

Attachment: Bug 1464834: Remove dead servo code. r=xidorn

Just leaving enough for style and tests to compile. Otherwise it's really
confusing, specially when looking for webidl stuff, and can confuse contributors
too.

I prefer to keep the rest as is, since the rest I do sync (semi) manually.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 8981155 [details]
Bug 1464834: Remove dead servo code. r=xidorn

Mozreview and splinter weren't happy about this. I basically left components/style + dependencies, ports/geckolib and tests/unit/style (which we don't run but should).

If you prefer it in try form there it is.

https://hg.mozilla.org/try/rev/bf2e325117b98c1a414ab444ebbe97e02e6159b9

Comment 3

[Phabricator Automation]

Comment on attachment 8981155 [details]
Bug 1464834: Remove dead servo code. r=xidorn

Xidorn Quan [:xidorn] UTC+10 has approved the revision.

https://phabricator.services.mozilla.com/D1434

Comment 4

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8981155 [details]
Bug 1464834: Remove dead servo code. r=xidorn

Reviewed in the phabricator.

Comment 5

[Cameron McCormack (:heycam)]

Comment on attachment 8981155 [details]
Bug 1464834: Remove dead servo code. r=xidorn

I can't log in to Phabricator at the moment, but r=me with some comments:

* There are some dotfiles under servo/ (including .cargo/) that can be deleted too.  (I think the important things from .hgignore / .gitignore are handled by the top level files but may be good to check.)

* Is it worth keeping servo/tests/unit/malloc_size_of/?

* I guess we are not currently running tidy.  Is it worth keeping servo/python/tidy/ and filing a bug to have it hooked up to the top-level mach?

Comment 6

[Emilio Cobos Álvarez (:emilio)]

(In reply to Cameron McCormack (:heycam) from comment #5)
> Comment on attachment 8981155 [details]
> Bug 1464834: Remove dead servo code. r=xidorn
> 
> I can't log in to Phabricator at the moment, but r=me with some comments:
> 
> * There are some dotfiles under servo/ (including .cargo/) that can be
> deleted too.  (I think the important things from .hgignore / .gitignore are
> handled by the top level files but may be good to check.)

Yeah, good point, got rid of them.

> * Is it worth keeping servo/tests/unit/malloc_size_of/?

We're not running them ATM, and they don't seem ultra-useful tests. I can keep them for now I suppose.

> * I guess we are not currently running tidy.  Is it worth keeping
> servo/python/tidy/ and filing a bug to have it hooked up to the top-level
> mach?

tidy can be installed via pip now, so I'd say even if we want to run tidy we don't want to keep the source there.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8fa010a5f187
Remove dead servo code. r=xidorn,heycam

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b6a31a23c372
Bring back servo/moz.build to fix the linting failure. r=me

Comment 9

[Phabricator Automation]

Comment on attachment 8981155 [details]
Bug 1464834: Remove dead servo code. r=xidorn

Cameron McCormack (:heycam) has been removed from the revision.

https://phabricator.services.mozilla.com/D1434

Comment 10

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/8fa010a5f187
https://hg.mozilla.org/mozilla-central/rev/b6a31a23c372