450362 - All files under media/ should have appropriate license headers

Status: RESOLVED WORKSFORME

(Core :: Audio/Video, defect)

Description

[Reed Loden [:reed]]

Seems some files under media/ are missing license headers... at least liboggplay_audio/sydney_audio.h is missing its license headers, but there may be more. Please check to make sure all files have appropriate license headers.

Comment 1

[Gervase Markham [:gerv]]

Currently, we are adopting the view that missing license headers means "same license as the code around it" - this is true of thousands of files of Mozilla code. I don't think it's worth creating divergences from upstream in 3rd party code just to add licensing headers. 

If you want to push the headers upstream and then repull, that would be great. :-) But it's not required. So I suggest WONTFIX.
 
Gerv

Comment 2

[Reed Loden [:reed]]

The code under liboggplay_audio/ seems to be tri-licensed, so it's not part of a third-party repo or code (we _are_ upstream), etc., so we should fix it.

Chris, correct me if I'm wrong, but I believe that you said this is not third-party code.

Comment 3

[Gervase Markham [:gerv]]

Whether it's third-party code or not, we don't _have_ to fix it. As long as some of the files have headers, and none have incorrect headers, I'm happy.

However, if it's Mozilla code, then I have no objection to it being fixed if someone wants to take the time to fix it.

Gerv

Comment 4

[cajbir (:cajbir)]

liboggplay_audio was the audio portion of Annodex's Firefox plugin project. Some of the code was contributed by me (the Alsa backend). They have recently (in the last week or so) split this audio code into a separate library called libsydneyaudio and I'll be updating things to reflect that.

Comment 5

[Reed Loden [:reed]]

libsydneyaudio/include/sydney_audio.h now has a header, so I think we're good for now...