After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 430228 - [sunaudio] copyright bug
[sunaudio] copyright bug
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.6
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-04-16 08:23 UTC by Brian Cameron
Modified: 2007-04-17 01:58 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
patch to clean up copyright info (5.20 KB, patch)
2007-04-16 08:24 UTC, Brian Cameron
committed Details | Review

Description Brian Cameron 2007-04-16 08:23:51 UTC
The copyright info in sunaudio plugin code isn't quite right.  It should include 2006 for all the plugins since we made many changes in 2006.  Also in gstsunaudiosrc.c the plugin attributed Wim Taymans and Erik Walthinsen, which was a copy-paste error.  They weren't involved with writing this code at all.  

Also remove David Schleef from gstsunaudiosrc.h - he wasn't involved with writing this plugin at all.  He only wrote the original sink plugin.  Note he wasn't ever mentioned in the gstsunaudiosrc.c file at all.

Also clean up the comments at the top of each file so they are a bit more consistant.
Comment 1 Brian Cameron 2007-04-16 08:24:17 UTC
Created attachment 86415 [details] [review]
patch to clean up copyright info
Comment 2 Brian Cameron 2007-04-16 08:25:03 UTC
Oh, also added Jan Schmidt to the gstsunaudiosrc.c plugin since he did a bunch of work to fix the code last December, 2006.
Comment 3 Tim-Philipp Müller 2007-04-16 21:29:54 UTC
The gstsunaudiosrc.h chunk of the patch seems to have been a reverse patch? 

Committed to CVS with that chunk reversed:

 2007-04-16  Tim-Philipp Müller  <tim at centricular dot net>

        Patch by: Brian Cameron  <brian.cameron at sun dot com>

        * sys/sunaudio/gstsunaudio.c:
        * sys/sunaudio/gstsunaudiomixer.c:
        * sys/sunaudio/gstsunaudiomixer.h:
        * sys/sunaudio/gstsunaudiomixerctrl.c:
        * sys/sunaudio/gstsunaudiomixerctrl.h:
        * sys/sunaudio/gstsunaudiomixertrack.h:
        * sys/sunaudio/gstsunaudiosink.c:
        * sys/sunaudio/gstsunaudiosink.h:
        * sys/sunaudio/gstsunaudiosrc.c:
        * sys/sunaudio/gstsunaudiosrc.h:
          Fix and/or update copyright attributions (#430228).

Comment 4 Brian Cameron 2007-04-17 01:58:23 UTC
Yes, sorry the gstsunaudiosrc.h was a reverse patch.  Thanks for fixing this.