Mailing List Archive: Bricolage: devel

[Bricolage-Devel] Re: [Bricolage-Commits] Moving to inst.

Index | Next | Previous | View Threaded

david at kineticode

Feb 18, 2002, 12:56 PM

Post #1 of 4 (716 views)
Permalink

[Bricolage-Devel] Re: [Bricolage-Commits] Moving to inst.

On Fri, 2002-02-15 at 07:40, Mark Jaroski wrote:
> Log Message:
> -----------
> Moving to inst.
>
> Removed Files:
> -------------
> /cvsroot/bricolage/bricolage/bin:
> Tag:
> Release_1_2
> bric_pggrant

I can think of no reason why this script should be a part of the
distribution. It should be run when preparing a distribution to create
the SQL script that grants the permissions, but it does not need to be
part of the distribution itself.

That's why I think it should be in distutils.

David

--
David Wheeler AIM: dwTheory
david [at] kineticode ICQ: 15726394
Yahoo!: dew7e
Jabber: Theory [at] jabber
Kineticode. Setting knowledge in motion.(sm)

_______________________________________________
Bricolage-Devel mailing list
Bricolage-Devel [at] lists
https://lists.sourceforge.net/lists/listinfo/bricolage-devel

david at kineticode

Feb 16, 2002, 6:13 PM

Post #2 of 4 (679 views)
Permalink

[Bricolage-Devel] Re: [Bricolage-Commits] Moving to inst. [In reply to]

mark at geekhive

Feb 19, 2002, 6:42 AM

Post #3 of 4 (677 views)
Permalink

Re: [Bricolage-Devel] Re: [Bricolage-Commits] Moving to inst. [In reply to]

David Wheeler wrote:
> I can think of no reason why this script should be a part of the
> distribution. It should be run when preparing a distribution to create
> the SQL script that grants the permissions, but it does not need to be
> part of the distribution itself.

The problem is that the build process depends on the file it
produces, which may not be around if this was checked out of
cvs and not in the distro. I suppose the counter argument
is that folks checking this out of cvs should use the
bric_pgimport script, but I think it's nice to have options,
and it doesn't really hurt anything to include this very
small script.

Otherwise I suppose I could just have make do the work
itself, but that would be harder since it doesn't have the
level of regex support that the script uses.

--
--
=================================================================
-- mark at geekhive dot net --

_______________________________________________
Bricolage-Devel mailing list
Bricolage-Devel [at] lists
https://lists.sourceforge.net/lists/listinfo/bricolage-devel

david at kineticode

Feb 19, 2002, 10:16 AM

Post #4 of 4 (672 views)
Permalink

Re: [Bricolage-Devel] Re: [Bricolage-Commits] Moving to inst. [In reply to]

On Tue, 2002-02-19 at 06:42, Mark Jaroski wrote:

> The problem is that the build process depends on the file it
> produces, which may not be around if this was checked out of
> cvs and not in the distro. I suppose the counter argument
> is that folks checking this out of cvs should use the
> bric_pgimport script, but I think it's nice to have options,
> and it doesn't really hurt anything to include this very
> small script.

Actually, bric_pgimport should go away, IMHO. It's simply a development
tool, and it's not at all portable. In that case, since you're putting
the script in inst, then I guess it makes sense for it to stay (although
we can have bric_prep delete it for a distribution). In that case, it
also makes sense that you'll need the code in bric_prep to concatenate
all the *.sql, *.val, and *.con files as well, correct?

Regards,

David

--
David Wheeler AIM: dwTheory
david [at] kineticode ICQ: 15726394
Yahoo!: dew7e
Jabber: Theory [at] jabber
Kineticode. Setting knowledge in motion.(sm)

_______________________________________________
Bricolage-Devel mailing list
Bricolage-Devel [at] lists
https://lists.sourceforge.net/lists/listinfo/bricolage-devel

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads