Re: Getting a Patch Included [CC-1009]

John Patrick <[email protected]>
Newsgroups gmane.comp.java.cruise-control.devel
Message-ID <[email protected]>
Take 2... my fault.

I was testing it on MacOSX, but indirectly using the example script below.

I tried it;
without a cc.pid
with an empty cc.pid
invalid pid in cc.pid
valid pid for bash in cc.pid

----
#!/usr/bin/env bash

## single instance per cc.pid
# 1) use ${CC_PID} is already defined
# 2) default ${PID} to -1 and attempt to read pid from ${CC_PID}
# 3) check ${PID} is not stale
if [ ! "${CC_PID}" ]; then
    CC_PID=cc.pid
fi
PID_INVALID="000000"
PID=${PID_INVALID}
if [ -f "${CC_PID}" ]; then
    PID=`head -1 ${CC_PID}`
    if [[ ${PID} != *[[:digit:]]*  ]]; then
        PID=${PID_INVALID}
    fi
fi
RUNNING=`ps -ef | grep "${PID}" | grep -v grep | grep bash | wc -l | sed "s|
||g"`
if [ "${RUNNING}" != "0" ]; then
    echo "CruiseControl already running"
    exit -1
fi

EXEC="echo hello"
echo $EXEC
$EXEC &
echo $! > ${CC_PID}
cat cc.pid
-----

John

On 19 December 2010 11:42, Dan Rollo <[email protected]> wrote:

> I was getting some odd errors when I tried to run some of the commands
> directly in a shell on Ubuntu. (Most likely due to my own mistakes).
> Just FMOI, what distro did you try the changes on?
>
> Thanks
> Dan
>
>
> On 12/19/2010 06:35 AM, Dan Rollo wrote:
> > Hi John,
> >
> > Posting the issue on the mailing list is the right thing to do.
> >
> > And yes, "committers pick them up as they are free."
> >
> > If it doesn't get picked up quickly, don't hesitate to nicely nag again
> > about it. ;)
> >
> > Dan
> >
> >
> > Message: 2
> > Date: Sun, 19 Dec 2010 08:47:23 +0000
> > From: John Patrick <[email protected]>
> > Subject: [Cruisecontrol-devel] Getting a Patch Included [CC-1009]
> > To: cruisecontrol-devel <[email protected]>
> > Message-ID:
> > <[email protected]>
> > Content-Type: text/plain; charset="utf-8"
> >
> > Hi,
> >
> > What is the process of getting a patch applied to subversion?
> >
> > I've created a patch that stops multiple instances of cruisecontrol using
> > the same ${CC_PID}, so effectively the same home directory.
> >
> > Who should I assign the issue to, or do committers pick them up as they
> are
> > free?
> >
> > John
>
>
>
> ------------------------------------------------------------------------------
> Lotusphere 2011
> Register now for Lotusphere 2011 and learn how
> to connect the dots, take your collaborative environment
> to the next level, and enter the era of Social Business.
> http://p.sf.net/sfu/lotusphere-d2d
> _______________________________________________
> Cruisecontrol-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/cruisecontrol-devel
>

------------------------------------------------------------------------------
Lotusphere 2011
Register now for Lotusphere 2011 and learn how
to connect the dots, take your collaborative environment
to the next level, and enter the era of Social Business.
http://p.sf.net/sfu/lotusphere-d2d

_______________________________________________
Cruisecontrol-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/cruisecontrol-devel
cc-1009.patch-r4568-take-2 (application/octet-stream, 3.7 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.