<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hey Max,<div class=""><br class=""></div><div class="">The mc metadata nightmare in git is fixed when you install GitFileTree-Mergedriver of Thierry Goubier [1].</div><div class=""><br class=""></div><div class="">I’m attending Grease at the moment and since the changes are minimal, I will try to fix the metadata merge conflicts by hand.</div><div class=""><br class=""></div><div class="">[1] <a href="https://github.com/ThierryGoubier/GitFileTree-MergeDriver" class="">https://github.com/ThierryGoubier/GitFileTree-MergeDriver</a></div><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 19 May 2017, at 16:27, Max Leske <<a href="mailto:maxleske@gmail.com" class="">maxleske@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" class=""><div class=""><br class="Apple-interchange-newline">On 18 May 2017, at 23:55, Johan Brichau <<a href="mailto:johan@inceptive.be" class="">johan@inceptive.be</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div dir="auto" class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hey Max,<div class=""><br class=""></div><div class="">I think a pull request on dev is required. </div><div class="">There is a more recent change on master but many commits are only on dev.</div><div class=""><br class=""></div><div class="">I just did not follow up very well after the last esug sprint what the state of seaside develop branch is… </div><div class="">But we should get those changes on master and release a new version. Since we require a new Grease version, a 3.3 is in order </div></div></div></div></blockquote><div class=""><br class=""></div><div class="">Done. I attempted to merge master back into develop but a git merge is a nightmare with all the meta data. The PR now only includes my changes. I've closed the original PR on master.</div><div class=""><br class=""></div><div class="">Cheers,</div><div class="">Max</div><br class=""><blockquote type="cite" class=""><div class=""><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div dir="auto" class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><div class=""><blockquote type="cite" class=""><div class="">On 18 May 2017, at 08:02, Max Leske <<a href="mailto:maxleske@gmail.com" class="">maxleske@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi,<div class=""><br class=""></div><div class="">I’ve opened one pull request on Seaside:master (develop is behind master, so I chose the most recent branch) and on on Grease:dev. I wasn’t sure about metadataless format for FileTree etc., so if you need me to do something differently let me know.</div><div class=""><br class=""></div><div class="">Cheers,</div><div class="">Max</div><div class=""><br class=""></div><div class=""><br class=""><div class=""><blockquote type="cite" class=""><div class="">On 17 May 2017, at 07:45, Johan Brichau <<a href="mailto:johan@inceptive.be" class="">johan@inceptive.be</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hey Max,<div class=""><br class=""></div><div class="">Perhaps you can make a pull request on the git repo?</div><div class=""><br class=""></div><div class="">thx for the work</div><div class="">Johan</div><div class=""><br class=""></div><div class=""><div class=""><blockquote type="cite" class=""><div class="">On 17 May 2017, at 06:49, Max Leske <<a href="mailto:maxleske@gmail.com" class="">maxleske@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div class=""></div><blockquote type="cite" class=""><div class=""><br class=""><blockquote type="cite" class="">On 16 May 2017, at 22:24, Esteban A. Maringolo <<a href="mailto:emaringolo@gmail.com" class="">emaringolo@gmail.com</a>> wrote:<br class=""><br class="">2017-05-16 17:20 GMT-03:00 Max Leske <<a href="mailto:maxleske@gmail.com" class="">maxleske@gmail.com</a>>:<br class=""><blockquote type="cite" class="">Hi Philippe,<br class=""><br class="">I’ve created an implementation, trying to do what you suggest; these are the changes:<br class=""><br class="">- removed WADynamicVariable<br class="">- added GRDynamicVariable to Grease-Core<br class=""></blockquote></blockquote><br class="">Sorry, this should have been Grease-Pharo30-Core.<br class=""><br class=""><blockquote type="cite" class=""><blockquote type="cite" class="">- subclasses of WADynamicVariable now inherit from GRDynamicVariable<br class="">- test cases for WADynamicVariable moved to Grease-Tests-Core (not sure about this. The tests should be the same for all platforms but Grease-Core does not contain GRDynamicVariable…)<br class=""></blockquote><br class="">Great! Thank you Max.<br class=""><br class="">Is Grease-*Dialect*-Core (e.g. Grease-Pharo-Core) a prerequisite of Grease-Core?<br class=""></blockquote><br class="">No, the other way around, according to the configuration.<br class=""><br class=""><blockquote type="cite" class=""><br class=""><br class=""><blockquote type="cite" class="">I don’t have write permissions on STHub, so I’m attaching the patches below.<br class=""></blockquote><br class="">As expected, the file didn't make it to the attachments of the mail :)<br class=""></blockquote><br class="">Especially because I forgot to attach them… :p</div><div class=""><br class=""></div><div class="">Here’s the link to a zip archive with the patch: <a href="https://www.dropbox.com/s/pgd6k12mwh92s06/GRDynamicVariable_patch.zip?dl=0" class="">https://www.dropbox.com/s/pgd6k12mwh92s06/GRDynamicVariable_patch.zip?dl=0</a>.<br class=""></div></blockquote><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br class=""></div><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Just remembered something I forgot to fix. New archive: <a href="https://www.dropbox.com/s/44smqfnx4cvplt3/GRDynamicVariable_patch2.zip?dl=0" class="">https://www.dropbox.com/s/44smqfnx4cvplt3/GRDynamicVariable_patch2.zip?dl=0</a>.</div><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br class=""></div><div class="" style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Max</div><br class=""><blockquote type="cite" class=""><div class=""><br class="">Cheers,<br class="">Max<br class=""><br class=""><blockquote type="cite" class=""><br class=""><blockquote type="cite" class="">I can also make the change for Squeak but I’m not sure how I would go about committing the change for VW / VASt. Let me know what you think.<br class=""></blockquote><br class=""><br class=""><br class=""><br class="">Esteban A. Maringolo<br class="">_______________________________________________<br class="">seaside-dev mailing list<br class=""><a href="mailto:seaside-dev@lists.squeakfoundation.org" class="">seaside-dev@lists.squeakfoundation.org</a><br class=""><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" class="">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br class=""></blockquote><br class=""></div></blockquote></div></div>_______________________________________________<br class="">seaside-dev mailing list<br class=""><a href="mailto:seaside-dev@lists.squeakfoundation.org" class="">seaside-dev@lists.squeakfoundation.org</a><br class=""><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" class="">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br class=""></div></blockquote></div><br class=""></div></div>_______________________________________________<br class="">seaside-dev mailing list<br class=""><a href="mailto:seaside-dev@lists.squeakfoundation.org" class="">seaside-dev@lists.squeakfoundation.org</a><br class=""><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" class="">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br class=""></div></blockquote></div><br class=""></div></div>_______________________________________________<br class="">seaside-dev mailing list<br class=""><a href="mailto:seaside-dev@lists.squeakfoundation.org" class="">seaside-dev@lists.squeakfoundation.org</a><br class=""><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" class="">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br class=""></div></blockquote></div><br class=""></div></div></div>_______________________________________________<br class="">seaside-dev mailing list<br class=""><a href="mailto:seaside-dev@lists.squeakfoundation.org" class="">seaside-dev@lists.squeakfoundation.org</a><br class=""><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" class="">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br class=""></div></blockquote></div><br class="" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">_______________________________________________</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">seaside-dev mailing list</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><a href="mailto:seaside-dev@lists.squeakfoundation.org" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">seaside-dev@lists.squeakfoundation.org</a><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a></div></blockquote></div><br class=""></div></body></html>