[libcamera-devel,v5,01/13] subprojects: Drop pybind11 from gitignore
diff mbox series

Message ID 20230603075615.20663-2-tomi.valkeinen@ideasonboard.com
State Accepted
Headers show
Series
  • py: New python bindings event handling
Related show

Commit Message

Tomi Valkeinen June 3, 2023, 7:56 a.m. UTC
As we no longer have pybind11 as a subproject, drop it from the
gitignore.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
---
 subprojects/.gitignore | 1 -
 1 file changed, 1 deletion(-)

Comments

Laurent Pinchart June 4, 2023, 8:47 a.m. UTC | #1
Hi Tomi,

Thank you for the patch.

On Sat, Jun 03, 2023 at 10:56:03AM +0300, Tomi Valkeinen wrote:
> As we no longer have pybind11 as a subproject, drop it from the
> gitignore.

This should have

Fixes: ad4719c10c6a ("py: Move to mainline pybind11 version")

> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>

While at it, you should drop
subprojects/packagefiles/pybind11/meson.build. I'll do so and update the
commit message to

subprojects: Drop leftovers of pybind11

The pybind11 subproject was removed in commit ad4719c10c6a ("py: Move to
mainline pybind11 version"), but two traces remained, one in .gitignore
and a second one as a meson.build for pybind11. Remove them.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  subprojects/.gitignore | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/subprojects/.gitignore b/subprojects/.gitignore
> index ebe59479..04b6271f 100644
> --- a/subprojects/.gitignore
> +++ b/subprojects/.gitignore
> @@ -4,4 +4,3 @@
>  /libyaml
>  /libyuv
>  /packagecache
> -/pybind11-*
Tomi Valkeinen June 4, 2023, 1:03 p.m. UTC | #2
On 04/06/2023 11:47, Laurent Pinchart wrote:
> Hi Tomi,
> 
> Thank you for the patch.
> 
> On Sat, Jun 03, 2023 at 10:56:03AM +0300, Tomi Valkeinen wrote:
>> As we no longer have pybind11 as a subproject, drop it from the
>> gitignore.
> 
> This should have
> 
> Fixes: ad4719c10c6a ("py: Move to mainline pybind11 version")
> 
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
> 
> While at it, you should drop
> subprojects/packagefiles/pybind11/meson.build. I'll do so and update the
> commit message to

Ah. So we keep finding these leftovers. I wonder if this one is the last =).

> subprojects: Drop leftovers of pybind11
> 
> The pybind11 subproject was removed in commit ad4719c10c6a ("py: Move to
> mainline pybind11 version"), but two traces remained, one in .gitignore
> and a second one as a meson.build for pybind11. Remove them.
> 
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Looks fine to me.

  Tomi

Patch
diff mbox series

diff --git a/subprojects/.gitignore b/subprojects/.gitignore
index ebe59479..04b6271f 100644
--- a/subprojects/.gitignore
+++ b/subprojects/.gitignore
@@ -4,4 +4,3 @@ 
 /libyaml
 /libyuv
 /packagecache
-/pybind11-*