[xen-tools] Re: More changes

[C.J. Adams-Collier]

I fixed all of the sev5 errors.  Here's a patch.  The unit tests pass
for create-xen-image, so I assume the fix is fine.  Also, I've
attached a ChangeLog file using the standard format.  Please add this
to the root of the project and add change notes on each submit.

I'm also working on re-factoring a lot of this code into a new
Xen::Tools CPAN module.  I'd be happy to host it in this repository,
but it looks like there's no root in which related projects live.

http://colliertech.org/~cjac/tmp/Xen-Tools-0.01.tar.gz

It's certainly not done yet, but you can see a bit of what I'm
thinking.  I'm starting with xen-create-image and moving the subs into
the Xen::Tools package.

Xen::Tools depends only on perl modules packaged for etch.

xt-create-xen-config is totally horked in CVS.  Unit tests are failing.

Cheers,

C.J.


On 8/30/07, Steve Kemp <steve at steve.org.uk> wrote:
> On Wed Aug 29, 2007 at 22:40:06 -0700, C.J. Adams-Collier wrote:
> > I am a fan of Perl::Critic.  I've added a rule to the Makefile to run
> > the perl scripts through Perl::Critic.  It relies on a perlcriticrc
> > file being in the same directory as the Makefile.  I've attached one.
>
>   Unfortunately perl critic module doesn't seem to be available as
>  a Debian package.  Once I've built one I'll try your patch and
>  see what it complains about!
>
> Steve
> --
> Debian GNU/Linux System Administration
> http://www.debian-administration.org/
>
>
>
>


-- 
moo.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 20070830-a.diff
Type: text/x-patch
Size: 13701 bytes
Desc: not available
URL: <http://sym.noone.org/pipermail/xen-tools-discuss/attachments/20070830/9bf5d201/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ChangeLog
Type: application/octet-stream
Size: 1304 bytes
Desc: not available
URL: <http://sym.noone.org/pipermail/xen-tools-discuss/attachments/20070830/9bf5d201/attachment.obj>