public inbox for ipfire-scm@lists.ipfire.org
 help / color / mirror / Atom feed
* [git.ipfire.org] IPFire 2.x development tree branch, master, updated. 0d774f0db56882073254973bd25da32c50f12801
@ 2024-12-14 12:07 Michael Tremer
  0 siblings, 0 replies; only message in thread
From: Michael Tremer @ 2024-12-14 12:07 UTC (permalink / raw)
  To: ipfire-scm

[-- Attachment #1: Type: text/plain, Size: 7553 bytes --]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "IPFire 2.x development tree".

The branch, master has been updated
       via  0d774f0db56882073254973bd25da32c50f12801 (commit)
       via  ca81c65144e64a54937a00bf79f7a51e80bc6d97 (commit)
       via  f2fce26932d86ac6e15ec33d58ae432958411410 (commit)
      from  804ac341a18718e5593ef8d77670d28018515aa7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 0d774f0db56882073254973bd25da32c50f12801
Author: Michael Tremer <michael.tremer(a)ipfire.org>
Date:   Sat Dec 14 12:05:47 2024 +0000

    flash-image: Create a journal when the filesystem is being created
    
    We recently started to have problems when a new installation was
    launched from the flash image that creating the journal corrupted the
    filesystem on the next mount operation.
    
    Since we would like all IPFire installations to have a journal, we
    create this now when we create the image and won't try to add it later.
    
    Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>

commit ca81c65144e64a54937a00bf79f7a51e80bc6d97
Author: Michael Tremer <michael.tremer(a)ipfire.org>
Date:   Sat Dec 14 12:05:46 2024 +0000

    installer: Remove the option to install without journal
    
    It was possible to install a new system without a journal. I think this
    is a very outdated concept now and should be avoided in favour of
    filesystem integrity.
    
    Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>

commit f2fce26932d86ac6e15ec33d58ae432958411410
Author: Michael Tremer <michael.tremer(a)ipfire.org>
Date:   Sat Dec 7 11:50:51 2024 +0000

    Config: Don't interpret file names as regular expressions
    
    When we are searching for changes in rootfiles, we walk through each
    file that we have found in the build and check if it exists in the
    rootfile. That check interpreted filenames as regular expressions which
    caused a problem in the case of "/usr/bin/[".
    
    This patch changes that grep will only search for an exact string match
    (-F) and the string must be the entire line (-x).
    
    Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>
    Reviewed-by: Adolf Belka <adolf.belka(a)ipfire.org>
    Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>

-----------------------------------------------------------------------

Summary of changes:
 lfs/Config                        |  6 +++---
 lfs/flash-images                  |  2 +-
 src/initscripts/system/partresize | 13 -------------
 src/installer/hw.c                |  5 -----
 src/installer/hw.h                |  7 +++----
 src/installer/main.c              |  1 -
 6 files changed, 7 insertions(+), 27 deletions(-)

Difference in files:
diff --git a/lfs/Config b/lfs/Config
index 1a59ebe1e..8547f69c6 100644
--- a/lfs/Config
+++ b/lfs/Config
@@ -292,13 +292,13 @@ define POSTBUILD
 		fi; \
 		sed -e "s/BUILDTARGET/$(BUILDTARGET)/g" -e "s/KVER/$(KVER)/g" -e "s/xxxMACHINExxx/$(BUILD_ARCH)/g" $$ROOTFILE > $(TARGET)_rootfile; \
 		while read -r line; do \
-			if grep -qG "^#$$line$$" $(TARGET)_rootfile; then echo "#$$line" >> $(TARGET); \
-			elif grep -qG "^$$line$$" $(TARGET)_rootfile ; then echo "$$line" >> $(TARGET); \
+			if grep -qFx "#$$line" $(TARGET)_rootfile; then echo "#$$line" >> $(TARGET); \
+			elif grep -qFx "$$line" $(TARGET)_rootfile ; then echo "$$line" >> $(TARGET); \
 			else echo "+$$line" >> $(TARGET); \
 			fi; \
 		done < $(TARGET)_diff; \
 		grep -v "^#" $(TARGET)_rootfile | while read -r line; do \
-			if ! grep -qG "^$$line$$" $(TARGET)_diff ; then echo "-$$line" >> $(TARGET); \
+			if ! grep -qFx "$$line" $(TARGET)_diff ; then echo "-$$line" >> $(TARGET); \
 			fi; \
 		done; \
 		rm -f $(TARGET)_rootfile; \
diff --git a/lfs/flash-images b/lfs/flash-images
index 2f3010fc4..b12a53da6 100644
--- a/lfs/flash-images
+++ b/lfs/flash-images
@@ -126,7 +126,7 @@ endif
 ifeq "$(EFI)" "1"
 	mkfs.vfat $(PART_EFI)
 endif
-	mkfs.ext4 -O ^has_journal,extent -F $(PART_ROOT)
+	mkfs.ext4 -F $(PART_ROOT)
 
 	# Most systems that use Flashimages has no RTC at boot
 	# so the interval check should disables
diff --git a/src/initscripts/system/partresize b/src/initscripts/system/partresize
index df9c43991..832bc7237 100644
--- a/src/initscripts/system/partresize
+++ b/src/initscripts/system/partresize
@@ -48,7 +48,6 @@ case "${1}" in
 			# Azure and Google Compute Platform
 			if running_on_ec2 || running_on_oci || running_on_azure || running_on_gcp; then
 				scon="on"
-				journal="on"
 			fi
 
 			mount /boot > /dev/null
@@ -110,18 +109,6 @@ case "${1}" in
 						root_dev="${dev::-2}"
 					fi
 
-					# Check if the device support smart
-					smartctl --smart=on "${root_dev}" > /dev/null
-					if [ ${?} = 0 ]; then
-						journal="on"
-					fi
-
-					# Enable journal
-					if [ "${journal}" = "on" ]; then
-						boot_mesg "Create journal on "${dev}" ..."
-						tune2fs -O has_journal "${dev}"
-					fi
-
 					boot_mesg "Growing root partition to maximum size..."
 					echo -e ',+' | sfdisk --no-reread -f -N${part_num} "${root_dev}" 2>/dev/null
 
diff --git a/src/installer/hw.c b/src/installer/hw.c
index 6bf05b185..577780af9 100644
--- a/src/installer/hw.c
+++ b/src/installer/hw.c
@@ -923,10 +923,6 @@ static int hw_format_filesystem(const char* path, int fs, const char* output) {
 	} else if (fs == HW_FS_EXT4) {
 		snprintf(cmd, sizeof(cmd), "/sbin/mke2fs -FF -T ext4 %s", path);
 
-	// EXT4 w/o journal
-	} else if (fs == HW_FS_EXT4_WO_JOURNAL) {
-		snprintf(cmd, sizeof(cmd), "/sbin/mke2fs -FF -T ext4 -O ^has_journal %s", path);
-
 	// XFS
 	} else if (fs == HW_FS_XFS) {
 		snprintf(cmd, sizeof(cmd), "/sbin/mkfs.xfs -f %s", path);
@@ -1027,7 +1023,6 @@ int hw_mount_filesystems(struct hw_destination* dest, const char* prefix) {
 	const char* filesystem;
 	switch (dest->filesystem) {
 		case HW_FS_EXT4:
-		case HW_FS_EXT4_WO_JOURNAL:
 			filesystem = "ext4";
 			break;
 
diff --git a/src/installer/hw.h b/src/installer/hw.h
index 92f32b67f..e1c3d345f 100644
--- a/src/installer/hw.h
+++ b/src/installer/hw.h
@@ -43,10 +43,9 @@
 
 #define HW_FS_SWAP                    0
 #define HW_FS_EXT4                    1
-#define HW_FS_EXT4_WO_JOURNAL         2
-#define HW_FS_XFS                     3
-#define HW_FS_FAT32                   4
-#define HW_FS_BTRFS                   5
+#define HW_FS_XFS                     2
+#define HW_FS_FAT32                   3
+#define HW_FS_BTRFS                   4
 
 #define HW_FS_DEFAULT                 HW_FS_EXT4
 
diff --git a/src/installer/main.c b/src/installer/main.c
index 9e3f4af83..9b9c630c3 100644
--- a/src/installer/main.c
+++ b/src/installer/main.c
@@ -644,7 +644,6 @@ int main(int argc, char *argv[]) {
 			char* description;
 		} filesystems[] = {
 			{ HW_FS_EXT4,            _("ext4 Filesystem") },
-			{ HW_FS_EXT4_WO_JOURNAL, _("ext4 Filesystem without journal") },
 			{ HW_FS_XFS,             _("XFS Filesystem") },
 			{ HW_FS_BTRFS,           _("BTRFS Filesystem (EXPERIMENTAL)") },
 			{ 0, NULL },


hooks/post-receive
--
IPFire 2.x development tree

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-12-14 12:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-12-14 12:07 [git.ipfire.org] IPFire 2.x development tree branch, master, updated. 0d774f0db56882073254973bd25da32c50f12801 Michael Tremer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox