Installing from CVS

Ugh.
I don’t know who’s in charge of the install.rb script, but whoever it is (and
future package manager programmers should listen too) - can we please have
the install scripts ignore any directories called ‘CVS’? Otherwise when I
checkout something and install it, I get all these CVS directories in my
/usr/local/lib/site_ruby/ directory, since the installer seems to install
everything it can get its hands on. I know I can do a ‘cvs export’ instead,
and I won’t get those directories, but that’s not so easy for updating the
archive - you have to re-export it every time, instead of just doing a ‘cvs
up’… I’m sure this would help developers, too.

Tim Bates

···


tim@bates.id.au

I don’t know who’s in charge of the install.rb script, but whoever
it is (and

If you mean the setup.rb/install.rb, Minero Aoki-san is. The contact
information can be found in the opening comment of the file itself.
(I’m CC’ing him this mail)

future package manager programmers should listen too) - can we please have
the install scripts ignore any directories called ‘CVS’? Otherwise when I
checkout something and install it, I get all these CVS directories in my
/usr/local/lib/site_ruby/ directory, since the installer seems to install
everything it can get its hands on. I know I can do a ‘cvs export’ instead,
and I won’t get those directories, but that’s not so easy for updating the
archive - you have to re-export it every time, instead of just doing a ‘cvs
up’… I’m sure this would help developers, too.

A quick fix is as follows:

Index: install.rb

···

At Sun, 9 Feb 2003 22:34:04 +0900, Tim Bates wrote:

RCS file: /src/shim/ruby16/install.rb,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
— install.rb 10 Dec 2002 13:34:17 -0000 1.8
+++ install.rb 28 Jan 2003 12:07:13 -0000 1.9
@@ -781,6 +781,7 @@
run_hook ‘pre-’ + task
send mid, rel.sub( %r_\A.*?(?:/|\z)_, ‘’ )
all_dirs( curr_srcdir ).each do |d|

  •      next if d == 'CVS'
         traverse task, rel + '/' + d, mid
       end
       run_hook 'post-' + task
    

I applied this to install.rb for use in the shim module. Aoki-san may
want to adopt this change into setup-3.1.2/src/base.rb.

Regards,


/
/__ __ Akinori.org / MUSHA.org
/ ) ) ) ) / FreeBSD.org / Ruby-lang.org
Akinori MUSHA aka / (_ / ( (__( @ iDaemons.org / and.or.jp

“It went right by me – At the time it went over my head
I was looking out the window… I should have looked at your face instead”

Hi,

In mail “Re: Installing from CVS”

I don’t know who’s in charge of the install.rb script, but whoever
it is (and

If you mean the setup.rb/install.rb, Minero Aoki-san is. The contact
information can be found in the opening comment of the file itself.
(I’m CC’ing him this mail)

Thank you. I have not know this thread…

A quick fix is as follows:

diff -u -r1.8 -r1.9
— install.rb 10 Dec 2002 13:34:17 -0000 1.8
+++ install.rb 28 Jan 2003 12:07:13 -0000 1.9
@@ -781,6 +781,7 @@
run_hook ‘pre-’ + task
send mid, rel.sub( %r_\A.*?(?:/|\z)_, ‘’ )
all_dirs( curr_srcdir ).each do |d|

  •      next if d == 'CVS'
         traverse task, rel + '/' + d, mid
       end
       run_hook 'post-' + task
    

I cannot accept this patch. Current setup.rb is designed for
only distribution/installation phase. If we use setup.rb for
development phase, we should re-design setup.rb. Orelse we
will encounter many problems.

  • Ignoring CVS/* is really sufficiant?
    setup.rb should ignore image files too.
  • setup.rb should ignore non-program files (e.g. resource text) too.

– Minero Aoki

···

“Akinori MUSHA” knu@iDaemons.org wrote:

At Sun, 9 Feb 2003 22:34:04 +0900, > Tim Bates wrote: