All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nayab Sayed via buildroot <buildroot@buildroot.org>
To: <buildroot@buildroot.org>
Cc: Nayab Sayed <nayabbasha.sayed@microchip.com>
Subject: [Buildroot] [PATCH] package/environment-setup: replace variable KERNELDIR with KERNEL_DIR
Date: Wed, 24 Jan 2024 11:02:21 +0530	[thread overview]
Message-ID: <20240124-kernel_dir-v1-1-4125408cf65d@microchip.com> (raw)

This commit updates the host environment setup helper script to use the more
conventional variable name KERNEL_DIR instead of KERNELDIR.  The change is made
for consistency and to align with established conventions in the project (like
KERNEL_HOST, HOST_DIR etc.). No functional changes are introduced by this
commit.

Signed-off-by: Nayab Sayed <nayabbasha.sayed@microchip.com>
---
 package/environment-setup/environment-setup    | 2 +-
 package/environment-setup/environment-setup.mk | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/environment-setup/environment-setup b/package/environment-setup/environment-setup
index 84a9843c24..8504a89743 100644
--- a/package/environment-setup/environment-setup
+++ b/package/environment-setup/environment-setup
@@ -10,7 +10,7 @@ cat <<'EOF'
 Some tips:
 * PATH now contains the SDK utilities
 * Standard autotools variables (CC, LD, CFLAGS) are exported
-* Kernel compilation variables (ARCH, CROSS_COMPILE, KERNELDIR) are exported
+* Kernel compilation variables (ARCH, CROSS_COMPILE, KERNEL_DIR) are exported
 * To configure do "./configure $CONFIGURE_FLAGS" or use
   the "configure" alias
 * To build CMake-based projects, use the "cmake" alias
diff --git a/package/environment-setup/environment-setup.mk b/package/environment-setup/environment-setup.mk
index 29ec5a9a95..cc058a5799 100644
--- a/package/environment-setup/environment-setup.mk
+++ b/package/environment-setup/environment-setup.mk
@@ -34,7 +34,7 @@ define HOST_ENVIRONMENT_SETUP_INSTALL_CMDS
 		>> $(ENVIRONMENT_SETUP_FILE)
 
 	$(if $(BR2_LINUX_KERNEL),\
-		printf "export \"KERNELDIR=$(LINUX_BUILDDIR)\"\n" \
+		printf "export \"KERNEL_DIR=$(LINUX_BUILDDIR)\"\n" \
 			>> $(ENVIRONMENT_SETUP_FILE),)
 endef
 

---
base-commit: 04dfeff6242516a1061973fc7af2cc9c5e3dd4e2
change-id: 20240124-kernel_dir-41a48fd97b75

Best regards,
-- 
Nayab Sayed <nayabbasha.sayed@microchip.com>

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

             reply	other threads:[~2024-01-24  5:42 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-24  5:32 Nayab Sayed via buildroot [this message]
2024-01-30 20:02 ` [Buildroot] [PATCH] package/environment-setup: replace variable KERNELDIR with KERNEL_DIR Arnout Vandecappelle via buildroot
2024-01-30 20:12   ` Angelo Compagnucci
2024-01-31 10:40     ` Nayab Sayed via buildroot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240124-kernel_dir-v1-1-4125408cf65d@microchip.com \
    --to=buildroot@buildroot.org \
    --cc=nayabbasha.sayed@microchip.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.