Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Apr 2017 16:30:39 +0000
From:      bugzilla-noreply@freebsd.org
To:        python@FreeBSD.org
Subject:   [Bug 203021] lang/python27: Install optional GDB integration tool(s)
Message-ID:  <bug-203021-21822-CsJA82eht7@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-203021-21822@https.bugs.freebsd.org/bugzilla/>
References:  <bug-203021-21822@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D203021

Conrad Meyer <cem@freebsd.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |https://reviews.freebsd.org
                   |                            |/D10398
           Keywords|needs-patch                 |

--- Comment #8 from Conrad Meyer <cem@freebsd.org> ---
(In reply to Kubilay Kocak from comment #7)
> Could you include your patch as an attachment please, or alternatively (p=
referred), open a Phabricator review (adding the review URl to the URl fiel=
d in bugzilla).

Done.

> Can you also confirm:
>=20
> - That the patch passes QA (poudriere)

Do we have any cluster machines that can be used for poudriere?  I'm moving=
 in
two days and my FreeBSD machines are packed.

> - To what extent this change should be made across all Python port versio=
ns. Ideally, include the changes for the other port versions in the patch/r=
eview as well

It should be done for all Python versions (and the patch does).

> - Whether this change (the addition of the file) should only be made when=
 the DEBUG port option is on, or unconditionally, with rationale if possibl=
e.

I don't see any reason to limit it to DEBUG.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-203021-21822-CsJA82eht7>