Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 802804 - net-misc/dhcp: use "${PN}" for last patch
Summary: net-misc/dhcp: use "${PN}" for last patch
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-18 22:01 UTC by Firas Khalil Khana
Modified: 2021-07-20 01:28 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Firas Khalil Khana 2021-07-18 22:01:40 UTC
In the `PATCHES=()` section all patches begin with `"${FILESDIR}/${PN}-` with the exception of the final patch `"${FILESDIR}/dhcp-`.
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-07-19 00:47:08 UTC
I agree it’s a minor style issue, but does it cause an actual problem for you..?
Comment 2 Firas Khalil Khana 2021-07-19 09:56:30 UTC
(In reply to Sam James from comment #1)
> I agree it’s a minor style issue, but does it cause an actual problem for
> you..?

I thank you for your quick reply.

It most certainly isn't a problem cause for me, and it definitely isn't a bug, so I was unsure how should I report similar stylistic issues.

Hopefully reporting them here isn't a bother?

Thanks again!
Comment 3 Larry the Git Cow gentoo-dev 2021-07-20 01:27:49 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7963dc6e972c419a8001ec70d852bcc519581eda

commit 7963dc6e972c419a8001ec70d852bcc519581eda
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2021-07-20 01:02:54 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2021-07-20 01:24:26 +0000

    net-misc/dhcp: don't hardcode PN in patch filename
    
    Match existing style.
    
    Closes: https://bugs.gentoo.org/802804
    Signed-off-by: Sam James <sam@gentoo.org>

 net-misc/dhcp/dhcp-4.4.2_p1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-07-20 01:28:44 UTC
(In reply to Firas Khalil Khana from comment #2)
> (In reply to Sam James from comment #1)
> > I agree it’s a minor style issue, but does it cause an actual problem for
> > you..?
> 
> I thank you for your quick reply.
> 
> It most certainly isn't a problem cause for me, and it definitely isn't a
> bug, so I was unsure how should I report similar stylistic issues.
> 
> Hopefully reporting them here isn't a bother?
> 
> Thanks again!

Sure, no problem at all, thanks for the report!

My only suggestion would be to either:
1) attach a patch (from git format-patch, with GCO sign off, so we can just apply it)
or
2) make a github PR

so it's easier to fix.