Hans,
Sure it would be tedious to have to type /tools each time you have to
start/stop an instance. But, like you proposed
symbolic links in root, there are means to avoid that on all platforms. I mean
it's easy to set something specific at the beginning
of a project and forget about what comes OOTB, being in dev, staging or prod.
So I'd not recommned adding symbolic links OOTB, this
would again defeat the purpose of the current goal: to slim down.
BTW I moved ofbiz.aptana.js.format.xml to tools. I think root is clean now.
Jacques
From: "Hans Bakker" <[email protected]>
On 03/25/2012 02:19 AM, Jacopo Cappellato wrote:
On Mar 24, 2012, at 12:32 PM, Hans Bakker wrote:
Hi Jacques,
Anything can be changed and anything can be adapted, however such basic
commands like start and stop should not be moved
lightly.
The compromise i can offer is to have the original commit with symbolic links
in the root called startofbiz.sh and stopofbiz.sh
We are not negotiating: please revert your commits because with them you are
blocking something that was discussed in the
community and agreed upon.
Jacopo
In order not to block further development i reverted the changes.
However , hiding such important changes in the many messages is not the way to
go and, as we also discussed, there should be a
proper description of the changes.
I left the original start/stop scripts in the root directory waiting until an
agreement is reached.
As i mentioned, my proposal is to convert these scripts to links into the tools
directory.
Hans
these commands are not used only by CI but much more often manually and i see
no reason to type every time tools/ in front of
it.
Regards,
Hans
On 03/24/2012 04:21 PM, Jacques Le Roux wrote:
From: "Hans Bakker"<[email protected]>
Sorry Jacopo, as I already stated, start/stop.sh and perhaps also the windows
ones should stay in the root.
I use Windows one, though not in for CI (I rely on Buildbot for that, too bad
those days). It was not a pb for me to add \tools
where necessary. I can't see that as a tremendous change and agree it should
not be reverted. Hans, naybe it's acceptable the
time you adapt your scripts though... We can't freeze OFBiz and have to move
ahead, based on consensus or vote, of course...
Jacques
They are operational-, not tools scripts.
further was looking if i could make them work in the root and then i saw they
were heavily modified without documentation so i
reverted also that.
Regards,
Hans
On 03/24/2012 12:42 PM, Jacopo Cappellato wrote:
On Mar 24, 2012, at 1:44 AM, Hans Bakker wrote:
AND make a number of changes not documented in the log message is not the way
to go
The "undocumented" changes are actually documented by the sentence in my log:
"Moved all platform dependent scripts to the
"tool" folder [...]"
In fact the purpose of the changes I did was to make them work in the new
location: I didn't add or change behavior and so I
have actually simply documented the commit with that sentence.
I agree I should have better specified what I did with something like:
"Moved all platform dependent scripts to the "tool" folder (and made required
changes to the scripts) [...]"
Jacopo
