- Pharo-dev - lists.pharo.org

[Pharo 7.0-dev] Build #532: 21312 Remove unnecessary "self run:" in ClassTraitTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #532 was: FAILURE. The Pull Request #844 was integrated: "21312 Remove unnecessary "self run:" in ClassTraitTest" Pull request url: https://github.com/pharo-project/pharo/pull/844 Issue Url: https://pharo.fogbugz.com/f/cases/21312 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

Do we remove TClass?
by Stephane Ducasse Feb. 12, 2018

Feb. 12, 2018

Hi Happy core developers do we remove TClass? Because it is duplicated code, no? Stef

2 2

[Pharo 7.0-dev] Build #531: 21297 Remove unnecessary "self run:" in SortedCollectionTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #531 was: SUCCESS. The Pull Request #859 was integrated: "21297 Remove unnecessary "self run:" in SortedCollectionTest" Pull request url: https://github.com/pharo-project/pharo/pull/859 Issue Url: https://pharo.fogbugz.com/f/cases/21297 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

Iceberg experience report - synchronising README change made on github
by Ben Coman Feb. 12, 2018

Feb. 12, 2018

Could the Iceberg gurus try and comment on the following workflow. Please familiarise with the first four commits here... https://github.com/Traadh/cloudflareun/network from these actions... "first commit" -- on github I created it as an unitialized repo to try to push a "new repo" to it per my other post, but when that failed from the command line I did `git init; git add README; git remote ...; git push ...`) "First Pharo code" -- cloned the repo with Iceberg, added a package, synchronised, added commit comment and clicked <Commit and push onto master> "Update class comment" -- edited the class comment, then from Iceberg synchronised, added commit comment and clicked <Commit and push onto master> "Updated README" -- on github, used the built-in editor to easily preview markdown. Clicked its <Commit directly to the master branch.> PROBLEM... Now I'm at a point having trouble synchronising Iceberg to pick up that last change. 1. Right-clicked on "cloudflareun" and chose "Synchronise repository..." 2. In [Update] tab, clicked <Fetch new versions> and listed under [commits to be imported] is "Updated README". Good. 3. Then clicked on <Pull> which I expected would load that commit into the Image, and a new commit "Merge with ref/remotes/origin/master" showed up in [commit to be imported] So now I'm lost. It seems the <Pull> only operates on the disk? But why does it create a merge commit? It should have fast-forwarded. Checking from the command line, only shows the first three commits are common with the github network graph. The fourth commit is the new merge commit. $ git log commit f49c64a7751f95712acc30dab692fc7e85e0c810 Merge: c818e9a 52ab6ac Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:55:34 2018 +0800 Merge with refs/remotes/origin/master commit c818e9adebeb6be001b202aa48da009be97920eb Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:42:57 2018 +0800 Update class comment. commit 6f2df89b219c56530412252ace96bc95eb78373a Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:37:01 2018 +0800 First Pharo code commit 52ab6ac696760a60fe56af57e45f2e2ceaaae35e Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:31:28 2018 +0800 first commit Checking the remote... $ git log origin/master commit 52ab6ac696760a60fe56af57e45f2e2ceaaae35e Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:31:28 2018 +0800 first commit Whoops... not what I expected but I don't think that affected anything now. Probably it was left over from early attempts to create a "new" "cloudflareun" repo. Try again... $ git remote --v origin git@github.com:Traadh/cloudflareun.git (fetch) origin git@github.com:Traadh/cloudflareun.git (push) traadh git@github.com:Traadh/cloudflareun.git (fetch) traadh git@github.com:Traadh/cloudflareun.git (push) $ git log traadh/master commit 587bed95c4b8fb3986c9962c9206405c81c99e3d Author: Ben Coman <github.com(a)ben.coman.com.au> Date: Thu Feb 8 11:54:20 2018 +0800 Updated README commit c818e9adebeb6be001b202aa48da009be97920eb Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:42:57 2018 +0800 Update class comment. commit 6f2df89b219c56530412252ace96bc95eb78373a Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:37:01 2018 +0800 First Pharo code commit 52ab6ac696760a60fe56af57e45f2e2ceaaae35e Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:31:28 2018 +0800 first commit So there are the four commits matching github. So it seems that Iceberg's FETCH did the right thing updating the remote. But the disk working directory should have just fast-forwarded rather than merging. Now seeing both these commits... * Merge with ref/remotes/origin/master * Updated README in the one [New commits to be imported] seems to be mixing results from the working repo and the remote repo in a way that makes it hard to guess the consequence of any next action here. I do see a <Merge and load> option when I right-click on one of those commits. Perhaps the "load" means to load from the disk working directory into the Image(?) but its a bit hidden compared to the <Fetch> and <Pull> that are buttons. It seems dangerous to take that action right now, since I guess the Image would diverge from the disk working directory. FIXING... What I ended up doing was remove the "merge commit" from the disk working directory $ git reset --hard HEAD^ then opened a new Iceberg which for [Updates] only showed only the single commit "Updated README" fetched into the remote repo. Good. I right-clicked on that and did <Merge and load> which correctly fast-forwarded the disk working directory... $ git log commit 587bed95c4b8fb3986c9962c9206405c81c99e3d Author: Ben Coman <github.com(a)ben.coman.com.au> Date: Thu Feb 8 11:54:20 2018 +0800 Updated README commit c818e9adebeb6be001b202aa48da009be97920eb Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:42:57 2018 +0800 Update class comment. commit 6f2df89b219c56530412252ace96bc95eb78373a Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:37:01 2018 +0800 First Pharo code commit 52ab6ac696760a60fe56af57e45f2e2ceaaae35e Author: Ben Coman <btc(a)openInWorld.com> Date: Thu Feb 8 11:31:28 2018 +0800 first commit However the same "Updated README" commit still shows in [New commits to be imported] tab, but now its right-click menu item now shows only <Load> rather than <Merge and load>. So it seem to correctly identify the disk working directory is up to date but clicking on <Load> doesn't change anything. Perhaps because there is no Pharo code to load? Since only the README outside the "src" directory changed? Moving to the [Push] tab, this shows the four commits seen in the last `git log` This is confusing since I'd expect this to be empty since... $ git log > /tmp/x1 $ git log traadh/master > /tmp/x2 $ diff /tmp/x1 /tmp/x2 is empty Luckily clicking <Publish all> cleared out this list with no impact on github network graph https://github.com/Traadh/cloudflareun/network SUMMARY In the end I seem to got done what I needed. I'm enjoying Iceberg more and more as I rub off the rough edges my understanding it. Hopefully my report will help smooth things for others. My only remaining niggle is not being able to get rid of this "1 incoming", but maybe that will disappear when I've got a code update to upload. cheers -ben P.S. It might be if the Iceberg [Remotes] could have a "Log" menu item that worked like `git log`. P.P.S. I think more thought needs to be given to whether mixing updates from both the remote and disk working directory in the one pane is beneficial? As an alternative, perhaps some side-scrolling GTInspector-like panes Image <--> Working Directory <--> Remote <--> Server(github) would better represent the git model. Or maybe statically position those panes in one window with buttons on directed lines showing which way and how far a commit moves when that button is pushed.

3 3

[Pharo 7.0-dev] Build #530: 21314 Remove unnecessary "self run:" in IntervalTest and IdentityBagTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #530 was: SUCCESS. The Pull Request #842 was integrated: "21314 Remove unnecessary "self run:" in IntervalTest and IdentityBagTest" Pull request url: https://github.com/pharo-project/pharo/pull/842 Issue Url: https://pharo.fogbugz.com/f/cases/21314 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 7.0-dev] Build #529: 21289 Remove unnecessary "self run:" in BasicBehaviorClassMetaclassTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #529 was: SUCCESS. The Pull Request #866 was integrated: "21289 Remove unnecessary "self run:" in BasicBehaviorClassMetaclassTest" Pull request url: https://github.com/pharo-project/pharo/pull/866 Issue Url: https://pharo.fogbugz.com/f/cases/21289 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

Esteban's ChangeLog week of 5 February 2018
by estebanlm＠gmail.com Feb. 12, 2018

Feb. 12, 2018

Hello! This is my weekly ChangeLog, from 5 February 2018 to 11 February 2018. You can see it in a better format by going here: http://log.smallworks.eu/web/search?from=5/2/2018&to=11/2/2018 ChangeLog ========= 6 February 2018: ---------------- * ... and packaging should be [restored](https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/214) too. Which means latest vm versions are latest builds from OSVM: === 64bit * [http://files.pharo.org/get-files/70/pharo64-mac-latest.zip](http://files.ph… * [http://files.pharo.org/get-files/70/pharo64-linux-threaded-latest.zip](http… * [http://files.pharo.org/get-files/70/pharo64-linux-latest.zip](http://files.… * [http://files.pharo.org/get-files/70/pharo64-win-latest.zip](http://files.ph… === 32bit * [http://files.pharo.org/get-files/70/pharo-win-latest.zip](http://files.phar… * [http://files.pharo.org/get-files/70/pharo-linux-threaded-latest.zip](http:/… * [http://files.pharo.org/get-files/70/pharo-linux-latest.zip](http://files.ph… * [http://files.pharo.org/get-files/70/pharo-mac-latest.zip](http://files.phar… === Hints * MacOS VMs now are signed! This should fix a lot of problems we had before. * Please take note that latest win64 build lacks all library dependencies, and many things will not work (working on that) Thanks to [Fabio](https://github.com/fniephaus) to help me put all the pieces together. * Certificates should work now, and we can have a [signed VM](https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/214). * yesterday with Fabio we worked on fix an upload of pharo-vm that was failing: * [https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/160](https://githu… 5 February 2018: ---------------- * I released a patch for iceberg: * [iceberg 0.6.7](https://github.com/pharo-vcs/iceberg/releases/tag/v0.6.7) * [libgit 1.0.5](https://github.com/pharo-vcs/libgit2-pharo-bindings/releases/tag/v1.… * [tonel 1.0.6](https://github.com/pharo-vcs/tonel/releases/tag/v1.0.6) this is a maintenance version that fixes a couple of bugs and cleans a bit, but big changes will come next release (0.7) * I published new versions of the certifictes to get a signed mac VM. In theory, this time it should work :) cheers! Esteban

1 0

[Pharo 7.0-dev] Build #528: 21322 LayoutFrameTest testIdentity should not use deprecated method fraction:
by ci-pharo-ci-jenkins2＠inria.fr Feb. 11, 2018

Feb. 11, 2018

There is a new Pharo build available! The status of the build #528 was: SUCCESS. The Pull Request #869 was integrated: "21322 LayoutFrameTest testIdentity should not use deprecated method fraction:" Pull request url: https://github.com/pharo-project/pharo/pull/869 Issue Url: https://pharo.fogbugz.com/f/cases/21322 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

flow to contribute?
by Stephane Ducasse Feb. 11, 2018

Feb. 11, 2018

Hi It looks like the flow to contribute changed. I do not find the fogbugz integration pop up anymore. Where is it? Was is changed? So I create a branch manually and commit in my fork and did a pull request but this is not clear. Stef

2 2

Iceberg package addition
by Stephane Ducasse Feb. 11, 2018

Feb. 11, 2018

Hi esteban each time I add a new packages to an iceberg project I get the same DNU KeyNotFound: why the name of my new package (which I added). packageNamed: packageName ^ self savedPackagesDictionary at: packageName ^^^^^^^^^^^^^^^ directoryForPackage: mcPackage | package | package := self packageNamed: mcPackage name. ^^^^^^^^^^^^^^^^^^^^ ^ self subdirectory isEmptyOrNil ifTrue: [ package directoryName ] ifFalse: [ self subdirectory, '/', package directoryName ] The second time I do it. It works but it does not give a good feeling. Stef

2 2