Monticello diff between image and tonel repo

Hi,

I think I already report it but I still see this issue. From Monticello
when I request changes between image and my tonel repo I got wrong diff
report.  It is a bit scarry. Will code be lost? You are not sure about
what really change.

Or do I misunderstand something?

See screenshot

Hilaire

--
Dr. Geo
http://drgeo.eu

Hi Ben,

The screenshot shows two things:

 1. In the top (black background), it shows the first lines of the file
    'DrGAngle3ptsOrientedItem.class.st' from the tonel repository, in
    the DrGeoII-Core tonel package. In this source, you see a
    'CostumeClasse' method.
 2. In the second half of the screenshot, in the Pharo environment,
    there are three windows to look at:
     1. in the background the Monticello window with the DrGeoII-Core
        package selected from the tonel repo (the same as in 1.);
     2. on top of the Monticello window, the repository is open and you
        can see its packages. The DrGeoII-Core package is selected;
     3. then the foreground window shows the changes between two
        packages: one is the tonel package, the other one I don't know
        (HilaireFernandes.7). Importantly, it look like costumClass
        method does not exist in the tonel package, which is wrong as
        seen in 1.

At the minimum, the information misrepresented, or I miss use the tool.
Anyway it is confusing for me or not obvious, likely more confusing for
newbie which may think the tool is unreliable, and they want to go away.

To reproduce, create a tonel repo, add package to this repo, or repo to
the package, and test the same as explained above.

Hilaire


Le 16/02/2019 à 07:23, Ben Coman a écrit :
> Its hard to determine what is wrong from the screen shot.
> I'm probably not the one able to provide an answer, 
> but anyway... can you provide steps-to-reproduce in a fresh Image?
>
--
Dr. Geo
http://drgeo.eu

Hi Ben,

I am not looking at you reproducing the scenario (you asked for) as it
is some work and the outcome may not be reliable as you mentioned it.
Nevertheless, the steps I described will likely get you on the trouble
and are easy to get in.

But more on sharing experience on this topic, as I meet it several
times. I think it is an important issue, I have since migrating to
Tonel, and it needs to be reported.

Closing Monticello windows and opening a fresh one helped to get the
changes right, sometime... Other time, forcing flush cash on Monticello
helps.

But still, this really makes fell the environment to be fragile and
inconstant for serious work. It seems Monticello is confused by two
sources of package tonel, and mcz. Already meet this problem too.

Hilaire


PS: DrGeo is free software.

Le 16/02/2019 à 10:52, Ben Coman a écrit :
> That isn't even close to "steps to reproduce".
>
> Thats just a lot of effort for me to produce scenario B that has a
> high chance of 
> being entirely unrelated to scenario A you presented.
>
--
Dr. Geo
http://drgeo.eu

I confirm again this issue, while annoying, flushing Monticello helps to
get a proper diff.

https://github.com/pharo-project/pharo/issues/2583

Hilaire

Le 16/02/2019 à 11:56, Hilaire a écrit :
> Closing Monticello windows and opening a fresh one helped to get the
> changes right, sometime... Other time, forcing flush cash on Monticello
> helps.

--
Dr. Geo
http://drgeo.eu