Minor typo in the paths in 3rd party library list

RESOLVED FIXED in Firefox 52

Status

defect
--
trivial
RESOLVED FIXED
3 years ago
Last year

People

(Reporter: psnyde2, Assigned: psnyde2, Mentored)

Tracking

Trunk
mozilla52
All
Other
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

(Whiteboard: [specification][type:bug], )

Attachments

(1 attachment)

Assignee

Description

3 years ago
What did you do?
================
Opened up tools/rewriting/ThirdPartyPaths.txt

What happened?
==============
I saw the path "media/kiss_ftt"

What should have happened?
==========================
This should be "media/kiss_fft"

Is there anything else we should know?
======================================
Assignee

Updated

3 years ago
Severity: normal → trivial
Component: Code Cleanup → General
Product: Mozilla Developer Network → Firefox
Version: unspecified → Trunk
Assignee

Updated

3 years ago
Summary: Minor typo in the paths in 3rd party library paths → Minor typo in the paths in 3rd party library list

Updated

3 years ago
Blocks: 1183143
Status: UNCONFIRMED → NEW
Component: General → Rewriting and Analysis
Ever confirmed: true
Product: Firefox → Core
This is trivial to fix, any volunteer? I can help with this.
Mentor: sledru
Assignee

Comment 2

3 years ago
Sure, if someone didn't mind doing a bit of handholding, I'd like to give it a whirl as a first big to fix. Is a patch submitted here good, or a branch and a merge request?  I did a quick tour of mdn and didn't see guidelines for submitting patches
(In reply to psnyde2 from comment #2)
> Sure, if someone didn't mind doing a bit of handholding, I'd like to give it
> a whirl as a first big to fix. Is a patch submitted here good, or a branch
> and a merge request?  I did a quick tour of mdn and didn't see guidelines
> for submitting patches
The protocol is to upload patches here, ask for a review and after that mark the patch for checkin-needed. A list with reviewers for each module can be found here: https://wiki.mozilla.org/Modules/All
Assignee

Updated

3 years ago
Attachment #8807925 - Attachment description: 1315438-correcting-third-party-paths.txt → Minor correction to the path for the third party media/kiss_fft path in the list of third party libraries in tools/rewriting/ThirdPartyPaths.txt
Attachment #8807925 - Attachment is patch: true
Assignee

Comment 5

3 years ago
(In reply to Andi-Bogdan Postelnicu from comment #3)
> (In reply to psnyde2 from comment #2)
> > Sure, if someone didn't mind doing a bit of handholding, I'd like to give it
> > a whirl as a first big to fix. Is a patch submitted here good, or a branch
> > and a merge request?  I did a quick tour of mdn and didn't see guidelines
> > for submitting patches
> The protocol is to upload patches here, ask for a review and after that mark
> the patch for checkin-needed. A list with reviewers for each module can be
> found here: https://wiki.mozilla.org/Modules/All

Thank you for cluing me in on the procedure.  This is good to go for a review then.  I don't see any formal way of requesting a review, so I'm hoping doing so in this text is sufficient.
(In reply to psnyde2 from comment #5)
> (In reply to Andi-Bogdan Postelnicu from comment #3)
> > (In reply to psnyde2 from comment #2)
> > > Sure, if someone didn't mind doing a bit of handholding, I'd like to give it
> > > a whirl as a first big to fix. Is a patch submitted here good, or a branch
> > > and a merge request?  I did a quick tour of mdn and didn't see guidelines
> > > for submitting patches
> > The protocol is to upload patches here, ask for a review and after that mark
> > the patch for checkin-needed. A list with reviewers for each module can be
> > found here: https://wiki.mozilla.org/Modules/All
> 
> Thank you for cluing me in on the procedure.  This is good to go for a
> review then.  I don't see any formal way of requesting a review, so I'm
> hoping doing so in this text is sufficient.

We're almost there, bu there are some things that need to be changed:

1. Add a reviewer to the patch by going to the patch details and in the review section add peer and mark it as r?
2. Here at Mozilla we have the patch title standardised as:
            [Bugzilla ID] - [Patch Description]
   For this patch in particular a good title is:
            Bug 1315438 - fix path for media/kiss_fft

For start a Michael - michael@thelayzells.com is a good reviewer for this patch.
Assignee

Updated

3 years ago
Attachment #8807925 - Attachment filename: 1315438-correcting-third-party-paths.txt → kiss_fft
Assignee

Updated

3 years ago
Attachment #8807925 - Attachment filename: kiss_fft → Bug 1315438 - fix path for media_kiss_fft
Attachment #8807925 - Flags: review+
Assignee

Updated

3 years ago
Attachment #8807925 - Flags: review+ → review?(michael)
Assignee

Updated

3 years ago
Attachment #8807925 - Attachment description: Minor correction to the path for the third party media/kiss_fft path in the list of third party libraries in tools/rewriting/ThirdPartyPaths.txt → Bug 1315438 - fix path for media/kiss_fft
Comment on attachment 8807925 [details] [diff] [review]
Bug 1315438 - fix path for media/kiss_fft

Review of attachment 8807925 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me! Thanks for the patch :)

When you're ready to check in the patch, add the keyword 'checkin-needed' under the tracking section to this bug, and a sheriff will come by and check it in for you.
Attachment #8807925 - Flags: review?(michael) → review+
Assignee

Updated

3 years ago
Keywords: checkin-needed
Assignee

Comment 8

3 years ago
Done, thanks much!
In the future, please make sure you've got Mercurial configured to generate patches with the proper commit information. The |./mach mercurial-setup| command automates the process, otherwise you can do so manually following the directions below:
https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Thanks for the patch!
Assignee: nobody → psnyde2

Comment 10

3 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a19077a2bd28
Fix the path to kiss_fft in the third party library list. r=mystor
Keywords: checkin-needed

Comment 11

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a19077a2bd28
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52

Updated

Last year
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.