Closed Bug 1461737 Opened 2 years ago Closed 11 months ago
Move nsstring-rs to a better location
47 bytes, text/x-phabricator-request
|Details | Review|
nsstring-rs currently lives within servo/ since it was used by both Servo and Gecko and there was syncing. Servo no longer syncs with Gecko, and no longer needs nsstring-rs, so it should be moved to a more logical location (perhaps next to xpcom-rs).
I figure this would be a pretty easy good-first-bug for someone?
Assignee: nika → nobody
The file nsstring -rs is at location mozilla-central/servo/support/gecko/nsstring/src/lib.rs and it should be moved to mozilla-central/xpcom/rust/xpcom/src folder. It this correct??
Hello, can I work on this bug?
(In reply to Sahil Bhosale from comment #2) > The file nsstring -rs is at location > mozilla-central/servo/support/gecko/nsstring/src/lib.rs and it should be > moved to mozilla-central/xpcom/rust/xpcom/src folder. > > It this correct?? Sorry for the delay in replying, I've been on PTO :-) The entire mozilla-central/servo/support/gecko/nsstring directory should be moved to mozilla-central/xpcom/rust/nsstring wholesale, and paths in various files (e.g. Cargo.toml) updated to reflect those changes.
Hello, I've decided to take this on as my first challenge and submitted the patch up for review. Built my first mach build, made the required changes and successfully built the second time. Looking forward to feedback :)
Assignee: nobody → bitnotri
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/8e900ef58b0f Move nsstring-rs to a better location, r=nika
You need to log in before you can comment on or make changes to this bug.