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 742316 - Remove the FSF postal address from copyright comments
Remove the FSF postal address from copyright comments
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2015-01-04 12:57 UTC by Jonas Danielsson
Modified: 2015-05-02 19:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Deleted references to postal address of FSF (31.14 KB, patch)
2015-04-12 21:56 UTC, Txus Ordorika
reviewed Details | Review
Delete references to FSF postal address (31.13 KB, patch)
2015-04-13 06:49 UTC, Txus Ordorika
accepted-commit_now Details | Review

Description Jonas Danielsson 2015-01-04 12:57:27 UTC
We do not want to track the postal address of the FSF instead we should rewrite them such as:

 * You should have received a copy of the GNU General Public License along
 * with GNOME Maps; if not, write to the Free Software Foundation,
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 

=>


* You should have received a copy of the GNU General Public License along
* with GNOME Maps; if not, see <http://www.gnu.org/licenses/>.
Comment 1 Txus Ordorika 2015-04-12 21:46:05 UTC
I assume the file that contains the GPL2 itself, COPYING, must track the address. Thus, not changed.
Comment 2 Txus Ordorika 2015-04-12 21:56:07 UTC
Created attachment 301433 [details] [review]
Deleted references to postal address of FSF

It's been replaced by gnu.org licenses page
Comment 3 Mattias Bengtsson 2015-04-13 00:24:40 UTC
Review of attachment 301433 [details] [review]:

The commit message should be in present tense, that is something like:

"Delete references to FSF postal address

Replace it with gnu.org licenses page"

...otherwise fine.

The patch looks good otherwise! Thanks! :)
Comment 4 Jonas Danielsson 2015-04-13 06:29:56 UTC
Wow, awesome work :) So many files! Didn't know what I asked of you.
I agree with Mattias comment, I can fix up the commit message while merging it, or you can update the patch. Your choice.

Thanks!
Comment 5 Txus Ordorika 2015-04-13 06:49:27 UTC
Created attachment 301437 [details] [review]
Delete references to FSF postal address

Replace it with gnu.org licenses page
Comment 6 Jonas Danielsson 2015-04-13 06:53:21 UTC
Review of attachment 301437 [details] [review]:

Thanks!

I will mark the other patch as obosolete!
Do you know about git-bz?

http://blog.fishsoup.net/2008/11/16/git-bz-bugzilla-subcommand-for-git/

It can help with this workflow!
Comment 7 Txus Ordorika 2015-04-13 06:56:11 UTC
I'm attaching the patches using git bz ^^.
Can be marked as obsolete a previous patch from git bz?
Comment 8 Jonas Danielsson 2015-04-13 06:58:41 UTC
nice!

Yes, it is automagical if the patch has the same subject line. In your case you would have needed to use the  'git bz attach -e' flag with your command and you would have been able to tell bugzilla to obsolete the previous patch.