patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
@ 2021-06-24  6:27 mohamad.noor.alim.hussin
  2021-06-25 12:34 ` Xueming(Steven) Li
  0 siblings, 1 reply; 4+ messages in thread
From: mohamad.noor.alim.hussin @ 2021-06-24  6:27 UTC (permalink / raw)
  To: stable; +Cc: Mohamad Noor Alim Hussin

From: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>

Finding with "librt" keyword would give the output
with full path of librt such as /usr/lib/librt.so
instead of -lrt.

Assume find_library() will prepend "lib", thus remove
"lib" from "librt" keyword. The output will shows as
-lrt.

Signed-off-by: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
---
 drivers/raw/ifpga/base/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/raw/ifpga/base/meson.build b/drivers/raw/ifpga/base/meson.build
index da2d6e33c..949f7f127 100644
--- a/drivers/raw/ifpga/base/meson.build
+++ b/drivers/raw/ifpga/base/meson.build
@@ -25,7 +25,7 @@ sources = [
 
 rtdep = dependency('librt', required: false)
 if not rtdep.found()
-	rtdep = cc.find_library('librt', required: false)
+	rtdep = cc.find_library('rt', required: false)
 endif
 if not rtdep.found()
 	build = false
-- 
2.32.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
  2021-06-24  6:27 [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library() mohamad.noor.alim.hussin
@ 2021-06-25 12:34 ` Xueming(Steven) Li
  2021-06-28  7:42   ` Hussin, Mohamad Noor Alim
  0 siblings, 1 reply; 4+ messages in thread
From: Xueming(Steven) Li @ 2021-06-25 12:34 UTC (permalink / raw)
  To: mohamad.noor.alim.hussin, stable

Hi Mohamad,

Is the patch merged into dpdk main branch? Need a line [ upstream commit <ID> ] in commit message.

BTW, have you run devtools/checkpatches.sh? Seems missing "Fixes: " line.

Best Regards,
Xueming

> -----Original Message-----
> From: stable <stable-bounces@dpdk.org> On Behalf Of mohamad.noor.alim.hussin@intel.com
> Sent: Thursday, June 24, 2021 2:28 PM
> To: stable@dpdk.org
> Cc: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> Subject: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
> 
> From: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> 
> Finding with "librt" keyword would give the output with full path of librt such as /usr/lib/librt.so instead of -lrt.
> 
> Assume find_library() will prepend "lib", thus remove "lib" from "librt" keyword. The output will shows as -lrt.
> 
> Signed-off-by: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> ---
>  drivers/raw/ifpga/base/meson.build | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/raw/ifpga/base/meson.build b/drivers/raw/ifpga/base/meson.build
> index da2d6e33c..949f7f127 100644
> --- a/drivers/raw/ifpga/base/meson.build
> +++ b/drivers/raw/ifpga/base/meson.build
> @@ -25,7 +25,7 @@ sources = [
> 
>  rtdep = dependency('librt', required: false)  if not rtdep.found()
> -	rtdep = cc.find_library('librt', required: false)
> +	rtdep = cc.find_library('rt', required: false)
>  endif
>  if not rtdep.found()
>  	build = false
> --
> 2.32.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
  2021-06-25 12:34 ` Xueming(Steven) Li
@ 2021-06-28  7:42   ` Hussin, Mohamad Noor Alim
  2021-06-28  7:52     ` Xueming(Steven) Li
  0 siblings, 1 reply; 4+ messages in thread
From: Hussin, Mohamad Noor Alim @ 2021-06-28  7:42 UTC (permalink / raw)
  To: Xueming(Steven) Li, stable

Hi Xueming,

This patch haven't merged in main branch. I will resend this patch to main branch.
I haven't run devtools/checkpatches.sh. Do I need to run this script before send the patch?

Regards,
Alim Hussin

-----Original Message-----
From: Xueming(Steven) Li <xuemingl@nvidia.com> 
Sent: Friday, June 25, 2021 8:34 PM
To: Hussin, Mohamad Noor Alim <mohamad.noor.alim.hussin@intel.com>; stable@dpdk.org
Subject: RE: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()

Hi Mohamad,

Is the patch merged into dpdk main branch? Need a line [ upstream commit <ID> ] in commit message.

BTW, have you run devtools/checkpatches.sh? Seems missing "Fixes: " line.

Best Regards,
Xueming

> -----Original Message-----
> From: stable <stable-bounces@dpdk.org> On Behalf Of mohamad.noor.alim.hussin@intel.com
> Sent: Thursday, June 24, 2021 2:28 PM
> To: stable@dpdk.org
> Cc: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> Subject: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
> 
> From: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> 
> Finding with "librt" keyword would give the output with full path of librt such as /usr/lib/librt.so instead of -lrt.
> 
> Assume find_library() will prepend "lib", thus remove "lib" from "librt" keyword. The output will shows as -lrt.
> 
> Signed-off-by: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> ---
>  drivers/raw/ifpga/base/meson.build | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/raw/ifpga/base/meson.build b/drivers/raw/ifpga/base/meson.build
> index da2d6e33c..949f7f127 100644
> --- a/drivers/raw/ifpga/base/meson.build
> +++ b/drivers/raw/ifpga/base/meson.build
> @@ -25,7 +25,7 @@ sources = [
> 
>  rtdep = dependency('librt', required: false)  if not rtdep.found()
> -	rtdep = cc.find_library('librt', required: false)
> +	rtdep = cc.find_library('rt', required: false)
>  endif
>  if not rtdep.found()
>  	build = false
> --
> 2.32.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
  2021-06-28  7:42   ` Hussin, Mohamad Noor Alim
@ 2021-06-28  7:52     ` Xueming(Steven) Li
  0 siblings, 0 replies; 4+ messages in thread
From: Xueming(Steven) Li @ 2021-06-28  7:52 UTC (permalink / raw)
  To: Hussin, Mohamad Noor Alim, stable

When patches accepted by main branch, normally LTS maintainer will scan all fixes for next LTS release.
You only need to send it manually if the patch expected to be included into LTS earlier, but first you need to get the patch merged into main branch.

Besides checkpatch, also need check-git-log,  please check official guide here:
	https://doc.dpdk.org/guides/contributing/patches.html

Thanks,
Xueming

> -----Original Message-----
> From: Hussin, Mohamad Noor Alim <mohamad.noor.alim.hussin@intel.com>
> Sent: Monday, June 28, 2021 3:43 PM
> To: Xueming(Steven) Li <xuemingl@nvidia.com>; stable@dpdk.org
> Subject: RE: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
> 
> Hi Xueming,
> 
> This patch haven't merged in main branch. I will resend this patch to main branch.
> I haven't run devtools/checkpatches.sh. Do I need to run this script before send the patch?
> 
> Regards,
> Alim Hussin
> 
> -----Original Message-----
> From: Xueming(Steven) Li <xuemingl@nvidia.com>
> Sent: Friday, June 25, 2021 8:34 PM
> To: Hussin, Mohamad Noor Alim <mohamad.noor.alim.hussin@intel.com>; stable@dpdk.org
> Subject: RE: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
> 
> Hi Mohamad,
> 
> Is the patch merged into dpdk main branch? Need a line [ upstream commit <ID> ] in commit message.
> 
> BTW, have you run devtools/checkpatches.sh? Seems missing "Fixes: " line.
> 
> Best Regards,
> Xueming
> 
> > -----Original Message-----
> > From: stable <stable-bounces@dpdk.org> On Behalf Of mohamad.noor.alim.hussin@intel.com
> > Sent: Thursday, June 24, 2021 2:28 PM
> > To: stable@dpdk.org
> > Cc: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> > Subject: [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library()
> >
> > From: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> >
> > Finding with "librt" keyword would give the output with full path of librt such as /usr/lib/librt.so instead of -lrt.
> >
> > Assume find_library() will prepend "lib", thus remove "lib" from "librt" keyword. The output will shows as -lrt.
> >
> > Signed-off-by: Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin@intel.com>
> > ---
> >  drivers/raw/ifpga/base/meson.build | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/raw/ifpga/base/meson.build b/drivers/raw/ifpga/base/meson.build
> > index da2d6e33c..949f7f127 100644
> > --- a/drivers/raw/ifpga/base/meson.build
> > +++ b/drivers/raw/ifpga/base/meson.build
> > @@ -25,7 +25,7 @@ sources = [
> >
> >  rtdep = dependency('librt', required: false)  if not rtdep.found()
> > -	rtdep = cc.find_library('librt', required: false)
> > +	rtdep = cc.find_library('rt', required: false)
> >  endif
> >  if not rtdep.found()
> >  	build = false
> > --
> > 2.32.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-06-28  7:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-24  6:27 [dpdk-stable] [PATCH 20.11] ifpga/meson: Fix finding librt using find_library() mohamad.noor.alim.hussin
2021-06-25 12:34 ` Xueming(Steven) Li
2021-06-28  7:42   ` Hussin, Mohamad Noor Alim
2021-06-28  7:52     ` Xueming(Steven) Li

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ https://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git