'Usability issue with external editors (1GE6IRL) [reply] Description Andre Weinand 2001-10-10 21:34:46 EDT - Setup a project that contains a *.gif resource - release project to CVS - edit the *.gif resource with an external editor, e.g. PaintShop - save and close external editor - in Navigator open the icon resource and verify that your changes are there - release project -> nothing to release! - in Navigator open the icon resource and verify that your changes are still there Problem: because I never "Refreshed from local", the workspace hasn\'t changed so "Release" didn\'t find anything. However opening the resource with an external editor found the modified file on disk and showed the changes. The real problem occurs if "Release" actually finds something to release but you don\'t spot that some resources are missing. This is extremely error prone: one of my changes didn\'t made it into build 110 because of this! NOTES: EG (5/23/01 3:00:33 PM) Release should do a refresh from local before doing the release. Moving to VCM KM (05/27/01 5:10:19 PM) Comments from JM in related email: Should not do this for free. Could have a setting which made it optoinal but should nt be mandatory. Default setting could be to have it on. Consider the SWT team who keep their workspaces on network drives. This will be slow. Side effects will be that a build runs when the refresh is completed unless you somehow do it in a workspace runnable and don\'t end the runnable until after the release. This would be less than optimal as some builders may be responsible for maintaining some invariants and deriving resources which are releasable. If you don\'t run the builders before releasing, the invariants will not be maintained and you will release inconsistent state. Summary: Offer to "ensure local consistency" before releasing. KM (5/31/01 1:30:35 PM) See also 1GEAG1A: ITPVCM:WINNT - Internal error comparing with a document which failed with an error. Never got log from Tod though. [reply] Comment 1 James Moody 2001-10-19 10:32:10 EDT *** Bug 183 has been marked as a duplicate of this bug. *** [reply] Comment 2 James Moody 2001-10-19 16:36:00 EDT Implemented \'auto refresh\' option. Default value is off. [reply] Comment 3 DJ Houghton 2001-10-23 23:39:03 EDT PRODUCT VERSION: 109 [reply] Comment 4 James Moody 2001-10-25 10:19:43 EDT Fixed in v206 [reply] Comment 5 Boris Bokowski 2006-11-01 16:25:53 EST I looked at this because of the link in Ian\'s blog (http://ianskerrett.wordpress.com/2006/11/01/looking-back-in-time-at-eclipse/). Much to my surprise, I can still reproduce the original issue with Eclipse 3.2.1! Why didn\'t we turn auto-refresh on by default? Does the SWT team still have their workspaces on network drives? [reply] Comment 6 John Arthorne 2006-11-01 16:35:52 EST Yes, people still use network drives. In fact, since in 3.2 we now allow projects to be backed by arbitrary file systems via EFS, there are even more scenarios where refresh will be expensive. Note also that James was referring to an "auto-refresh" option in the sync view. This was replaced with a global auto-refresh in the 3.0 release. This is still off by default, because as mentioned before, it can be expensive. [reply] Comment 7 Boris Bokowski 2006-11-01 16:42:41 EST (In reply to comment #6) > This is still off by default, because as mentioned before, it can be expensive. Couldn\'t we turn it on by default now that we have jobs, and a place where jobs show up in the UI? If the refresh happened in a background job that was displayed in the status line, people would have a way to find out what\'s going on and disable it if they don\'t like it. [reply] Comment 8 John Arthorne 2006-11-01 17:22:38 EST Boris: see bug 89672 [reply] Comment 9 Philippe Ombredanne 2006-11-09 14:03:39 EST Happy birthday bug 1, you are five years old, and still kicking :-D [reply] Comment 10 John Arthorne 2006-11-09 14:17:41 EST Closing. [reply] Comment 11 Eclipse Genie 2015-05-19 05:30:50 EDT New Gerrit change created: https://git.eclipse.org/r/48136 [reply] Comment 12 Eclipse Genie 2015-05-19 06:41:33 EDT Gerrit change https://git.eclipse.org/r/48136 was merged to [master]. Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=980cf72c9ea237e5f896bc0cc74ec2b2dc05ccf5 [reply] Comment 13 Denis Roy 2015-05-19 09:47:26 EDT (In reply to Eclipse Genie from comment #11) > New Gerrit change created: https://git.eclipse.org/r/48136 I will fix her. Add Comment Collapse All Comments Expand All Comments'