Manuscript (Case [Issue]22749) Iceberg (Git) - Iceberg does not see rename of extended class which leads to broken commit

Classic

List

Threaded

♦

♦

Locked 1 message

Pharo Issue Tracker

Reply | Threaded

Open this post in threaded view

♦

♦

|

Manuscript (Case [Issue]22749) Iceberg (Git) - Iceberg does not see rename of extended class which leads to broken commit

Denis Kudryashov opened Case 22749: Iceberg does not see rename of extended class which leads to broken commit and assigned it to Esteban Lorenzano: Bug in  Iceberg (Git): Misc  •  You are subscribed to this case I detected problem during query renames in Calypso. So to reproduce problem first set up Calypso repository: Metacello new baseline: 'Calypso'; repository: 'github://pharo-ide/Calypso:dev/src'; onUpgrade: [:ex | ex useIncoming]; load Then try rename ClyUndeclaredMethods to ClyUndeclaredMethodsQuery and open commit window from iceberg. You will not see changes related to the package which extends this class (Calypso-SystemPlugins-Undeclared-Browser). So if you will just commit without noticing problem you will get broken code. Priority: 2 – Really Important Status: Work Needed Assigned to: Esteban Lorenzano Milestone: Pharo7.0 Go to Case No longer need updates? Unsubscribe from this case. Don't want Manuscript notifications anymore? Update your preferences. A Manuscript case was opened by Denis Kudryashov. Case ID: 22749 Title: Iceberg does not see rename of extended class which leads to broken commit Status: Work Needed Category: Bug Project: Iceberg (Git) Area: Misc Priority: 2 - Really Important Milestone: Pharo7.0: 02.09.2018 (Past) Assigned To: Esteban Lorenzano URL: https://pharo.manuscript.com/f/cases/22749 I detected problem during query renames in Calypso. So to reproduce problem first set up Calypso repository: Metacello new baseline: 'Calypso'; repository: 'github://pharo-ide/Calypso:dev/src'; onUpgrade: [:ex | ex useIncoming]; load Then try rename ClyUndeclaredMethods to ClyUndeclaredMethodsQuery and open commit window from iceberg. You will not see changes related to the package which extends this class (Calypso-SystemPlugins-Undeclared-Browser). So if you will just commit without noticing problem you will get broken code. You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.manuscript.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=22749) from this case. _______________________________________________ Pharo-bugtracker mailing list [hidden email] https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker