Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 532368 - [arch-tools] file-stabilization-bugs.py shouldn't set URL
Summary: [arch-tools] file-stabilization-bugs.py shouldn't set URL
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Paweł Hajdan, Jr. (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 532054
  Show dependency tree
 
Reported: 2014-12-12 12:17 UTC by Pacho Ramos
Modified: 2014-12-18 20:21 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
0002-Do-not-set-the-URL-for-a-stabilization-request.patch (0002-Do-not-set-the-URL-for-a-stabilization-request.patch,1.22 KB, patch)
2014-12-14 17:39 UTC, William Hubbs
Details | Diff
0002-Do-not-set-the-URL-for-a-stabilization-request.patch (0002-Do-not-set-the-URL-for-a-stabilization-request.patch,1.28 KB, patch)
2014-12-18 19:56 UTC, William Hubbs
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pacho Ramos gentoo-dev 2014-12-12 12:17:47 UTC
I needed to change this one year ago as some people complained about setting "URL" without needing :/
$ diff -u /usr/bin/file-stabilization-bugs file-stabilization-bugs.py
--- /usr/bin/file-stabilization-bugs	2014-12-11 16:11:07.272770929 +0100
+++ file-stabilization-bugs.py	2013-12-24 19:30:13.783799355 +0100
@@ -75,14 +75,12 @@
 			description = ('Is it OK to stabilize =%s ?\n' % cpv +
 				       'If so, please CC all arches which have stable keywords\n' +
 				       'for older versions of this package.')
-			url = 'http://packages.gentoo.org/package/%s?arches=linux' % urllib.quote(cp)
 			params = {}
 			params['product'] = 'Gentoo Linux'
 			params['version'] = 'unspecified'
 			params['component'] = 'Keywording and Stabilization'
 			params['summary'] = '%s: stabilization request' % cpv
 			params['description'] = description
-			params['url'] = url
 			params['assigned_to'] = maintainer
 			params['cc'] = other_maintainers
 			params['severity'] = 'enhancement'
Comment 1 William Hubbs gentoo-dev 2014-12-14 17:39:19 UTC
Created attachment 391714 [details, diff]
0002-Do-not-set-the-URL-for-a-stabilization-request.patch

This patch drops the code that sets the URL for a stabilization request.
Comment 2 William Hubbs gentoo-dev 2014-12-18 19:56:40 UTC
Created attachment 391980 [details, diff]
0002-Do-not-set-the-URL-for-a-stabilization-request.patch

This is an updated version of this patch, mark the other one obsolete.

Thanks,

William
Comment 3 Pacho Ramos gentoo-dev 2014-12-18 20:07:19 UTC
(In reply to William Hubbs from comment #2)
> Created attachment 391980 [details, diff] [details, diff]
> 0002-Do-not-set-the-URL-for-a-stabilization-request.patch

Works nice :D
Comment 4 William Hubbs gentoo-dev 2014-12-18 20:21:47 UTC
Per Pacho's request, I am merging this into bug #532054; there will be
one patch that covers all of this.