Hello,
Also it is rather strange that there is no space between your name and mail address. Did you do something like the above : git repo-config user.name "Your Name" git repo-config user.email "me@example.com"
silvio@port1024.net wrote:
From: Silvio fricke<silvio@port1024.net>
Very strange, because [1] show me that I have spaces between my name and address.
I have only minor "appearance" issues to address here, I don't know what to think about the patch itself. Only that it does not seem too big and mostly concentrated in one place, which is good.
Ok, fine!
So first, isn't the correct term "splitted"? It seems that you use both "split" and "splittet" which would then be incorrect.
Hmm, english is not my native language and I need help in language things. But I have the suggestions from [2] used.
Also you should read this : http://archlinux.org/pacman/submitting-patches.html Especially this part : << Describe your patch.
Before the actual diff begins, it helps if you describe the changes in the patch. This allows others to see what you intended so as to compare it to what was actually done, and allows better feedback. If you use git-format-patch to create your patch, then your commit message will be shown above the patch by default.
Right you are! The next version of this patch has a more descriptive description.
+ if [ "$LOGGING" = "1" ]; then + [ "x$BUILDLOG" == "x" ]&& BUILDLOG="${startdir}/${pkgname}-${pkgver}-${pkgrel}-${CARCH}.log" + echo>> "$BUILDLOG" + echo>> "$BUILDLOG" + echo "--- Logging for $it ------------">> "$BUILDLOG" +
What about wraiden's suggestion in the bug comments : << You could change the comment appending into a single line of the following form which would be cleaner in my opinion.
echo -e "\n\n--- Logging for $it ------------" >> "$BUILDLOG"
I have it beautified! [1] http://archlinux.org/pipermail/pacman-dev/2008-June/012133.html [2] http://archlinux.org/pipermail/pacman-dev/2008-June/012147.html Bye eS.eF. -- -- S. Fricke ---------------------------------------- silvio@port1024.net -- Diplom-Informatiker (FH) Linux-Entwicklung JABBER: silvio@conversation.port1024.net ----------------------------------------------------------------------------