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 "[EMAIL PROTECTED]"

> [EMAIL PROTECTED] wrote:
> > From: Silvio fricke<[EMAIL PROTECTED]>

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 ---------------------------------------- [EMAIL PROTECTED] --
   Diplom-Informatiker (FH)
   Linux-Entwicklung             JABBER: [EMAIL PROTECTED]
----------------------------------------------------------------------------


_______________________________________________
pacman-dev mailing list
pacman-dev@archlinux.org
http://archlinux.org/mailman/listinfo/pacman-dev

Reply via email to