Skip to contentSkip to conversation

tools/resolve_btfids: add /libbpf to .gitignore

PATCH Applied Stanislav Fomichev 4 participants 6 messages Feb 12, 2021

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 tools/resolve_btfids: add /libbpf to .gitignore

There is what I see after compiling the kernel:

 # bpf-next...bpf-next/master
 ?? tools/bpf/resolve_btfids/libbpf/
---
 tools/bpf/resolve_btfids/.gitignore | 1 +
 1 file changed, 1 insertion(+)
tools/bpf/resolve_btfids/.gitignore +1 -0
index 25f308c933cc..16913fffc985 100644
@@ -1,2 +1,3 @@
 /fixdep
 /resolve_btfids
+/libbpf/
Signature · 1 line
2.30.0.478.g8a0d178c01-goog
#1 lore raw
Quoted 2 lines · Stanislav Fomichev

There is what I see after compiling the kernel:

typo: This?

Quoted 6 lines
 # bpf-next...bpf-next/master
 ?? tools/bpf/resolve_btfids/libbpf/

---

Jiri,

Is this

Fixes fc6b48f692f8 ("tools/resolve_btfids: Build libbpf and libsubcmd

in separate directories")

?

Do we need similar stuff for libsubcmd (what's that, btw?)

Quoted 14 lines
 tools/bpf/resolve_btfids/.gitignore | 1 +
 1 file changed, 1 insertion(+)
tools/bpf/resolve_btfids/.gitignore +1 -1
index 25f308c933cc..16913fffc985 100644
@@ -1,2 +1,3 @@
 /fixdep
 /resolve_btfids
+/libbpf/
--

2.30.0.478.g8a0d178c01-goog

#2 lore raw
Quoted 6 lines · Andrii Nakryiko
Quoted 2 lines · Stanislav Fomichev

There is what I see after compiling the kernel:

typo: This?

Yes, sure.

Quoted 8 lines
Quoted 5 lines
 # bpf-next...bpf-next/master
 ?? tools/bpf/resolve_btfids/libbpf/

---

Jiri,

Quoted 8 lines

Is this

Fixes fc6b48f692f8 ("tools/resolve_btfids: Build libbpf and libsubcmd

in separate directories")

?

Do we need similar stuff for libsubcmd (what's that, btw?)

It's probably not needed because it has only .o files in there (.o are ignored in the root .gitignore).
I assume libbpf/ has an issue because there is bpf_helper_defs.h in that libbpf/ directory.
Not sure why it was removed in fc6b48f692f8 rather than being prefixed with libbpf/ directory.
I'll leave it up to Jiri to comment.

Quoted 14 lines
Quoted 14 lines
 tools/bpf/resolve_btfids/.gitignore | 1 +
 1 file changed, 1 insertion(+)
tools/bpf/resolve_btfids/.gitignore +1 -1
index 25f308c933cc..16913fffc985 100644
@@ -1,2 +1,3 @@
 /fixdep
 /resolve_btfids
+/libbpf/
--

2.30.0.478.g8a0d178c01-goog

#3 lore raw
Quoted 26 lines · Stanislav Fomichev
Quoted 6 lines · Andrii Nakryiko
Quoted 2 lines · Stanislav Fomichev

There is what I see after compiling the kernel:

typo: This?

Yes, sure.

Quoted 15 lines
Quoted 5 lines
 # bpf-next...bpf-next/master
 ?? tools/bpf/resolve_btfids/libbpf/

---

Jiri,

Is this

Fixes fc6b48f692f8 ("tools/resolve_btfids: Build libbpf and libsubcmd

in separate directories")

?

Quoted 8 lines
Quoted 1 line

Do we need similar stuff for libsubcmd (what's that, btw?)

It's probably not needed because it has only .o files in there (.o are ignored in the root .gitignore).
I assume libbpf/ has an issue because there is bpf_helper_defs.h in that libbpf/ directory.
Not sure why it was removed in fc6b48f692f8 rather than being prefixed with libbpf/ directory.
I'll leave it up to Jiri to comment.

you're right, thanks for the fix

jirka

Quoted 17 lines
Quoted 16 lines
Quoted 16 lines

 tools/bpf/resolve_btfids/.gitignore | 1 +
 1 file changed, 1 insertion(+)
tools/bpf/resolve_btfids/.gitignore +1 -1
index 25f308c933cc..16913fffc985 100644
@@ -1,2 +1,3 @@
 /fixdep
 /resolve_btfids
+/libbpf/
--

2.30.0.478.g8a0d178c01-goog

#4 lore raw
In reply to Andrii Nakryiko
Quoted 13 lines · Andrii Nakryiko
Quoted 2 lines · Stanislav Fomichev

There is what I see after compiling the kernel:

typo: This?

Quoted 6 lines
 # bpf-next...bpf-next/master
 ?? tools/bpf/resolve_btfids/libbpf/

---

Quoted 6 lines

Jiri,

Is this

Fixes fc6b48f692f8 ("tools/resolve_btfids: Build libbpf and libsubcmd

in separate directories")

yes

Quoted 4 lines

?

Do we need similar stuff for libsubcmd (what's that, btw?)

as Stanislav said it's only .o files in there, so there's no need

it provides the parse_options functionality

jirka

Quoted 16 lines
Quoted 15 lines
 tools/bpf/resolve_btfids/.gitignore | 1 +
 1 file changed, 1 insertion(+)
tools/bpf/resolve_btfids/.gitignore +1 -1
index 25f308c933cc..16913fffc985 100644
@@ -1,2 +1,3 @@
 /fixdep
 /resolve_btfids
+/libbpf/
--

2.30.0.478.g8a0d178c01-goog

#5 lore raw
In reply to Stanislav Fomichev
This patch was applied to bpf/bpf-next.git (refs/heads/master):

Hello:

This patch was applied to bpf/bpf-next.git (refs/heads/master):

Quoted 8 lines · On Thu, 11 Feb 2021 17:00:53 -0800 you wrote:

There is what I see after compiling the kernel:

 # bpf-next...bpf-next/master
 ?? tools/bpf/resolve_btfids/libbpf/

[...]

Here is the summary with links:
- [bpf-next] tools/resolve_btfids: add /libbpf to .gitignore

    https://git.kernel.org/bpf/bpf-next/c/90a82b1fa40d

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html