Page 1 of 1

[SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-10T11:14:05-07:00
by NicolasRobidoux
I unfortunately deleted the directory tree I had, two hours ago, so I can't directly compare, but the code inside pixel.c is definitely outdated. It predates the changes to bicubic which introduced Catmull-Rom.

I'll be double checking that I did not so something silly, but I figured this was urgent to let the info out just in case something bad happened.

RE: svn of IM7 reverted to an older version???

Posted: 2012-06-10T11:20:38-07:00
by NicolasRobidoux
Indeed:
svn co https://www.imagemagick.org/subversion/ ... gick-7.0.0 ImageMagick
brings about a version of pixel.c which looks like it is several versions old, version that predates switching bicubic to Catmull-Rom and probably several other things.

Re: svn of IM7 reverted to an older version???

Posted: 2012-06-10T11:22:32-07:00
by NicolasRobidoux
svn trunk seems fine.

Re: svn of IM7 reverted to an older version of pixel.c

Posted: 2012-06-10T11:26:26-07:00
by NicolasRobidoux
I put a copy of something which should be pretty close to what should be IM7 pixel.c here: http://web.cs.laurentian.ca/nrobidoux/m ... ed_pixel.c

Re: svn of IM7 reverted to an older version of pixel.c

Posted: 2012-06-10T11:29:13-07:00
by NicolasRobidoux
svn log shows a jump from r7174 to r8212.

Re: svn of IM7 reverted to an older version of pixel.c

Posted: 2012-06-10T11:36:07-07:00
by NicolasRobidoux
I made one single commit last week into 7.0.0. Could I have broken something? I svn ci right back to the web location from which I had svn co the source tree using
svn ci -m "mext -> next typo".

Re: svn of IM7 reverted to an older version of pixel.c

Posted: 2012-06-10T12:06:08-07:00
by NicolasRobidoux
The log of svn trunk (IM6) does not show a large jump.
I am quite certain that the IM7 svn is broken.

Re: [URGENT] IM7 svn broken: lost months of commits

Posted: 2012-06-10T12:23:14-07:00
by magick
Development is in the Subversion trunk. Try
Make your changes there and commit. Its the alpha development branch and all ImageMagick-7.0.0 alpha distribution releases are built from there.

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-10T13:00:03-07:00
by NicolasRobidoux
So, I may have been the one who broke things by commiting back into https://www.imagemagick.org/subversion/ ... gick-7.0.0.
I'll stick to https://subversion.imagemagick.org/subv ... dows/trunk from now on.

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-10T13:11:25-07:00
by NicolasRobidoux
magick: I know what went wrong: I've never used branches with svn (git only), and consequently completely misunderstood you when you wrote
Nicolas, don't forget you need to interpolate alpha separately and blend it into the color channel results. Here's the refactored code. You / Anthony can update this method in the Subversion trunk. Don't forget there are 4 locations. Two places in ImageMagick 7.0.0-0 and two in ImageMagick 6.7.7-6.
On top of it, I had forgotten that the "true" subversion repo is not the "public" one.
Apologies.

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-10T14:00:12-07:00
by NicolasRobidoux
magick: You really want me to svn co and ci through ImageMagick-Windows or you actually want me to do it through the 7.0.0 and 6.7.7 branches of https://subversion.imagemagick.org/subv ... /branches/ (without merging back into trunk), or just touch https://subversion.imagemagick.org/subv ... ick/trunk/ or something else?
(Still reading about how to deal with multiple branches.)

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-10T15:18:53-07:00
by magick
For the ImageMagick 7 alpha series, use
Go to ImageMagick/ImageMagick and make your changes and commits from there. Periodically use
  • svn update
to add any changes Anthony / Glenn / Cristy makes.

For the ImageMagick 6 stable series, use
go to ImageMagick-6.7.7 and make your changes and commits from there. Periodically use
  • svn update
to add any changes Anthony / Glenn / Cristy makes.

In general, be very careful in the ImageMagick 6 series because its the stable release. You can be less careful in the ImageMagick 7 series because its alpha and is expected to break. Of course, don't break it if you can avoid it.

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-10T15:36:44-07:00
by NicolasRobidoux
magick wrote:For the ImageMagick 7 alpha series, use
Aahhh! You really do mean the ImageMagick-Windows trunk then, not the ImageMagick trunk, just inside the ImageMagick folder instead of at the top level.
I would never have guessed.

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-11T16:57:58-07:00
by anthony
You can see what changes are between your copy and the SVN using

Code: Select all

 svn st
For commits I use

Code: Select all

svn st -q
to see what files I have changed (I never commit everything due to some personal changes)

Remember before you can commit you need to merge in the 'updates' from the SVN, and resolve any conflicts.

Re: [SOLVED] IM7 svn broken: lost months of commits

Posted: 2012-06-12T08:39:28-07:00
by NicolasRobidoux
For the record, here is how I committed the changes to the one file I changed, pixel.c.
First I created an svn repo in my home dir:

Code: Select all

svn co https://subversion.imagemagick.org/subversion/ImageMagick-Windows/trunk ImageMagick
All I wanted to change at this point was the pixel.c file in ImageMagick-Windows/trunk/ImageMagick/MagickCore, which is actually in ~/ImageMagick/ImageMagick/MagickCore. I cd'd there and

Code: Select all

svn changelist INTERPOLATE pixel.c
Edited, double checked etc. Committed the changes with

Code: Select all

svn commit --changelist INTERPOLATE -m "pixel.c: clean up Catrom and fix Spline"