GSoC 2024 [PATCH]

GSoC 2024 [PATCH]

[Aishwarya Narayanan]

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

Dear Git Organization,

I hope this email finds you well. I've prepared a patch to address an
issue related to the handling of exit codes in our test suite
concerning the Microproject idea Avoid suppressing Git’s exit code in
test scripts. The changes ensure that the exit code of the write_utf16
and write_utf32 functions is properly captured and not suppressed,
improving the reliability of our testing process.
I kindly request your review and any remarks or changes you may have.
Please accept my apologies for any mistakes or oversights. Thank you
for your time and attention.

Best regards,
 Aishwarya Narayanan

[-- Attachment #2: t0028-working-tree-encoding.patch --]
[-- Type: application/octet-stream, Size: 1509 bytes --]

From 1493c834b2373226992d6c58f88549e7bc8dfd40 Mon Sep 17 00:00:00 2001
From: Aishwarya Narayanan <aishnana.03@gmail.com>
Date: Wed, 27 Mar 2024 14:47:40 +0530
Subject: [PATCH] The change ensures that the exit code of  write_utf16 and
 write_utf32 is properly captured and not suppressed.

---
 t/t0028-working-tree-encoding.sh | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/t/t0028-working-tree-encoding.sh b/t/t0028-working-tree-encoding.sh
index ad151a3467..f372c14bc0 100755
--- a/t/t0028-working-tree-encoding.sh
+++ b/t/t0028-working-tree-encoding.sh
@@ -19,10 +19,13 @@ test_expect_success 'setup test files' '
 	echo "*.utf16 text working-tree-encoding=utf-16" >.gitattributes &&
 	echo "*.utf16lebom text working-tree-encoding=UTF-16LE-BOM" >>.gitattributes &&
 	printf "$text" >test.utf8.raw &&
-	printf "$text" | write_utf16 >test.utf16.raw &&
-	printf "$text" | write_utf32 >test.utf32.raw &&
-	printf "\377\376"                         >test.utf16lebom.raw &&
-	printf "$text" | iconv -f UTF-8 -t UTF-16LE >>test.utf16lebom.raw &&
+	test_utf16_raw=$(printf "$text" | write_utf16) &&
+	test_utf32_raw=$(printf "$text" | write_utf32) &&
+	test_utf16lebom_raw=$(printf "\377\376"; printf "$text" | iconv -f UTF-8 -t UTF-16LE) &&
+	echo "$test_utf16_raw" > test.utf16.raw &&
+	echo "$test_utf32_raw" > test.utf32.raw &&
+	echo "$test_utf16lebom_raw" > test.utf16lebom.raw &&
+
 
 	# Line ending tests
 	printf "one\ntwo\nthree\n" >lf.utf8.raw &&
-- 
2.44.0.windows.1

Re: GSoC 2024 [PATCH]

[Patrick Steinhardt]

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

On Wed, Mar 27, 2024 at 03:05:57PM +0530, Aishwarya Narayanan wrote:
> Dear Git Organization,
> 
> I hope this email finds you well. I've prepared a patch to address an
> issue related to the handling of exit codes in our test suite
> concerning the Microproject idea Avoid suppressing Git’s exit code in
> test scripts. The changes ensure that the exit code of the write_utf16
> and write_utf32 functions is properly captured and not suppressed,
> improving the reliability of our testing process.
> I kindly request your review and any remarks or changes you may have.
> Please accept my apologies for any mistakes or oversights. Thank you
> for your time and attention.

Please not that patches are supposed to be sent as mails directly
instead of being attached to mails. You can use e.g. git-send-email(1)
to achieve this.

I would recommend to have a look "Documentation/MyFirstContribution.txt"
and "Documentation/SubmittingPatches". These resources should get you a
better picture of how a typical workflow would look like here.

Thanks!

Patrick

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: GSoC 2024 [PATCH]

[Junio C Hamano]

Patrick Steinhardt <ps@pks.im> writes:

> Please not that patches are supposed to be sent as mails directly
> instead of being attached to mails. You can use e.g. git-send-email(1)
> to achieve this.
>
> I would recommend to have a look "Documentation/MyFirstContribution.txt"
> and "Documentation/SubmittingPatches". These resources should get you a
> better picture of how a typical workflow would look like here.

Thanks.  It is very good that you recommended the documentation on
the contribution process.

The tasks for microproject are designed to be easy to solve and the
changes in the patches themselves would not be the interesting part
for new contributors.  Rather, it is designed for a new contributor
to experience the contribution process from the beginning to the
end, from coming up with an initial patch and explaining it well,
sending it to the list in the established way, receiving and
responding to review comments, and move on to the next iteration.