On Fri, Dec 11, 2015 at 12:52 PM, Eliot Miranda <[email protected]> wrote:
> Hi Mariano, > > for this in Squeak I added a simple menu pick to the Monticello > Browser's package list menu called "search history" which opens up a > workspace containing the entire history as a flat string. I then just > search using ctrl-f. You can find it in a current squeak trunk image. > > Hi Eliot, Yes, I was thinking the same. In the meanwhile, I did this extension: MCWorkingCopy >> gtInspectorIrIn: composite <gtInspectorPresentationOrder: 30> composite text title: 'History'; display: [ :workingCopy | TimeProfiler spyOn: [| stream versionInfos | stream := String new writeStream. versionInfos := workingCopy ancestry withBreadthFirstAncestors select: [:each | each isKindOf: MCVersionInfo]. versionInfos do: [ :each | stream nextPutAll: each summary; cr; cr. ]. stream contents.]. ] And then I can use the regular cmd+f to search the string. But I have to manually inspect the package doing something like: (MCWorkingCopy allManagers detect: [:each | each packageName = 'MyPackage']) inspect. > _,,,^..^,,,_ (phone) > > On Dec 10, 2015, at 9:12 AM, Mariano Martinez Peck <[email protected]> > wrote: > > How many times have you tried to search a string as a comment of a commit > in all the history? Myself: many many times. I know we need much better > tools for that, store the history in a another way, provide a nicer API, > etc etc. And I know there were topics about that. > > However, I would deeply appreciate a very short term solution for ease > that. The model side is as simple as this: > > | packageName ancestry anscestors substring | > packageName := 'MyPackageXX'. > substring := 'whatever I want to search'. > ancestry := (MCWorkingCopy allManagers detect: [:each | each packageName = > packageName] ) ancestry. > anscestors := ancestry withBreadthFirstAncestors select: [:each | each > isKindOf: MCVersionInfo]. > anscestors select: [ :each | each message includesSubstring: substring ] > > Do you think it's worth a simple UI tool? Custom inspector of MCAncestry? > > Cheers, > > > -- > Mariano > http://marianopeck.wordpress.com > > -- Mariano http://marianopeck.wordpress.com
