[SOLVED] IM7 svn broken: lost months of commits

Post any defects you find in the released or beta versions of the ImageMagick software here. Include the ImageMagick version, OS, and any command-line required to reproduce the problem. Got a patch for a bug? Post it here.
Post Reply
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

[SOLVED] IM7 svn broken: lost months of commits

Post 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.
Last edited by NicolasRobidoux on 2012-06-10T12:57:20-07:00, edited 6 times in total.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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.
Last edited by NicolasRobidoux on 2012-06-10T11:24:02-07:00, edited 1 time in total.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post by NicolasRobidoux »

svn trunk seems fine.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post by NicolasRobidoux »

svn log shows a jump from r7174 to r8212.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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".
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post by NicolasRobidoux »

The log of svn trunk (IM6) does not show a large jump.
I am quite certain that the IM7 svn is broken.
User avatar
magick
Site Admin
Posts: 11064
Joined: 2003-05-31T11:32:55-07:00

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

Post 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.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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.)
User avatar
magick
Site Admin
Posts: 11064
Joined: 2003-05-31T11:32:55-07:00

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

Post 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.
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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.
User avatar
anthony
Posts: 8883
Joined: 2004-05-31T19:27:03-07:00
Authentication code: 8675308
Location: Brisbane, Australia

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

Post 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.
Anthony Thyssen -- Webmaster for ImageMagick Example Pages
https://imagemagick.org/Usage/
NicolasRobidoux
Posts: 1944
Joined: 2010-08-28T11:16:00-07:00
Authentication code: 8675308
Location: Montreal, Canada

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

Post 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"
Post Reply