Open
Bug 946619
Opened 12 years ago
Updated 3 years ago
use mozpack in libvpx update.py
Categories
(Core :: WebRTC: Audio/Video, defect)
Tracking
()
NEW
backlog | parking-lot |
People
(Reporter: j, Unassigned)
References
Details
Some parts of update.py could be improved using mozpack.files
Reporter | ||
Updated•12 years ago
|
Reporter | ||
Updated•12 years ago
|
Summary: use mozpack.files in libvpx update.py → use mozpack in libvpx update.py
Reporter | ||
Comment 1•12 years ago
|
||
mozpack.copier.FileCopier should also be used to copy files
Reporter | ||
Comment 2•12 years ago
|
||
> @@ +376,5 @@
> > + sources_mozbuild_new = "files = {\n %s\n}\n" % f.read().strip()[1:-1]
> > + if sources_mozbuild != sources_mozbuild_new:
> > + print 'updating sources.mozbuild'
> > + with open('sources.mozbuild', 'w') as f:
> > + f.write(sources_mozbuild_new)
>
> You can use mozbuild.util.FileAvoidWrite instead of passing around the
> previous sources_mozbuild content.
Updated•10 years ago
|
Component: Audio/Video → WebRTC: Audio/Video
Comment 4•10 years ago
|
||
The feature request is still a reasonable idea, but I don't think anyone's interested in working on it. I'm fine with closing it.
Flags: needinfo?(giles)
Comment 5•10 years ago
|
||
Thanks. I'm moving this to "parking-lot" which means it's a recognized feature request, but unprioritized and unscheduled. We'll take patches for it.
backlog: --- → parking-lot
Updated•10 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•