# [PATCH] Insert ACTIVESTATE_STRING in Git.pm

4 messages from 2007-01-22 to 2007-01-22. Participants: Alex Riesen, Petr Baudis.
Thread: https://gitlist.dev/t/6467

## Alex Riesen, 2007-01-22 14:32

Subject: [PATCH] Insert ACTIVESTATE_STRING in Git.pm
Message-ID: <81b0412b0701220632t50ad42c4ucb0ebae140a9706a@mail.gmail.com>
URL: https://gitlist.dev/e/81b0412b0701220632t50ad42c4ucb0ebae140a9706a%40mail.gmail.com

```
Also add "git" to the pipe parameters, otherwise it does not work at all, as
no git commands are usable out of git context.

---

This is not enough to make it work on ActiveState. It's just to
fix obvious TODOs, which should help someone to get it going.
The other problems are:
 - tie refuses to bind a handle class activestate_pipe to a scalar $fh
   (a patch follows)
 - there are some unexplained warnings about uninitialized values
 - cygwin-windows pathname conversion (not fixable at all)

 perl/Git.pm |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

```

## Alex Riesen, 2007-01-22 14:43

Subject: [PATCH] Insert ACTIVESTATE_STRING in Git.pm
Message-ID: <81b0412b0701220643s3d337664jcddfffd35071b08e@mail.gmail.com>
URL: https://gitlist.dev/e/81b0412b0701220643s3d337664jcddfffd35071b08e%40mail.gmail.com
In-Reply-To: <81b0412b0701220632t50ad42c4ucb0ebae140a9706a@mail.gmail.com>

```
Also add "git" to the pipe parameters, otherwise it does not work at all, as
no git commands are usable out of git context.

---

Oh, fine, just what I needed... Forgot to attach the patch!

> This is not enough to make it work on ActiveState. It's just to
> fix obvious TODOs, which should help someone to get it going.
> The other problems are:
>  - tie refuses to bind a handle class activestate_pipe to a scalar $fh
>    (a patch follows)
>  - there are some unexplained warnings about uninitialized values
>  - cygwin-windows pathname conversion (not fixable at all)
>
>  perl/Git.pm |    7 ++++---
>  1 files changed, 4 insertions(+), 3 deletions(-)
>

```

## Petr Baudis, 2007-01-22 15:44

Subject: Re: [PATCH] Insert ACTIVESTATE_STRING in Git.pm
Message-ID: <20070122154429.GG18276@pasky.or.cz>
URL: https://gitlist.dev/e/20070122154429.GG18276%40pasky.or.cz
In-Reply-To: <81b0412b0701220632t50ad42c4ucb0ebae140a9706a@mail.gmail.com>

```
On Mon, Jan 22, 2007 at 03:32:28PM CET, Alex Riesen wrote:
> Also add "git" to the pipe parameters, otherwise it does not work at all, as
> no git commands are usable out of git context.
> 
> ---
> 
> This is not enough to make it work on ActiveState. It's just to
> fix obvious TODOs, which should help someone to get it going.
> The other problems are:
> - tie refuses to bind a handle class activestate_pipe to a scalar $fh
>   (a patch follows)
> - there are some unexplained warnings about uninitialized values
> - cygwin-windows pathname conversion (not fixable at all)
> 
> perl/Git.pm |    7 ++++---
> 1 files changed, 4 insertions(+), 3 deletions(-)

Cool.

Acked-by: Petr Baudis <pasky@suse.cz>

Perhaps ACPIPE could use some comment in the code but that's just vain
nitpicking.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
The meaning of Stonehenge in Traflamadorian, when viewed from above, is:
"Replacement part being rushed with all possible speed."
		-- Kurt Vonnegut, Sirens from Titan

```

## Alex Riesen, 2007-01-22 16:07

Subject: Re: [PATCH] Insert ACTIVESTATE_STRING in Git.pm
Message-ID: <81b0412b0701220807x284bcbfeo7e67599424451587@mail.gmail.com>
URL: https://gitlist.dev/e/81b0412b0701220807x284bcbfeo7e67599424451587%40mail.gmail.com
In-Reply-To: <20070122154429.GG18276@pasky.or.cz>

```
On 1/22/07, Petr Baudis <pasky@suse.cz> wrote:
> On Mon, Jan 22, 2007 at 03:32:28PM CET, Alex Riesen wrote:
> > Also add "git" to the pipe parameters, otherwise it does not work at all, as
> > no git commands are usable out of git context.
> >
> > ---
> >
> > This is not enough to make it work on ActiveState. It's just to
> > fix obvious TODOs, which should help someone to get it going.
> > The other problems are:
> > - tie refuses to bind a handle class activestate_pipe to a scalar $fh
> >   (a patch follows)
> > - there are some unexplained warnings about uninitialized values
> > - cygwin-windows pathname conversion (not fixable at all)
> >
> > perl/Git.pm |    7 ++++---
> > 1 files changed, 4 insertions(+), 3 deletions(-)
>
> Cool.
>
> Acked-by: Petr Baudis <pasky@suse.cz>
>
> Perhaps ACPIPE could use some comment in the code but that's just vain
> nitpicking.

Will do, with a sign-off this time.

```