[OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

akgrant43
 

Ok, don't mind the branch name. This started out as a trial to improve OpenBSD support, especially since my SqueakSSL/Unix changes broke it for them. But things got out of hand (viz. Yak shaving )

Travis Changes

  • This adds a few more libraries for Travis, such that more plugins are actually built and not disabled due to missing dependencies

scripts/updateSCCSVersions

  • This is a bash script now, but bash is not in /usr/bin on all platforms. Solution: use /usr/bin/env bash. Problem 2: We want bash -e, but POSIX/execve only specifies #! <interpreter> <onearg>. Solution 2:

    #! /usr/bin/env bash
    set -e

    @fniephaus we should do that for all instances of bash -e, probably.

Autoconf.

  • A lot of selections, flags, and specifics where handled in dispersed mvm files.
  • Most of which can now be handled via the results of autoconf, that is
    • the Makefiles (with their respective flags)
    • the config.h
  • Also, modernized a good bunch of autoconf/m4 files to more recent styles.
  • And, finally, selection of OpenSSL/LibreSSL backend settings for SqueakSSL via configure.

Missing here

  • not all acinclude.m4 adhere to autoconf's current style.
  • Cog's options (eg, COGMTVM) are not yet covered.

Tiny things

  • Improve no-op definotion of VM_LABEL as not to make gcc complain about strange expressions.

You can view, comment on, or merge this pull request online at:

  https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226

Commit Summary

  • compatibility
  • Split
  • on autoconf.
  • more autoconf
  • typo
  • Tiny improvements
  • newlines...
  • reorder
  • fix pulse
  • cosmetics
  • uuid
  • Remove hardcoded -luuid
  • autoconf best practice
  • pthread governed by configure.ac
  • not using this
  • idem
  • apparently needed
  • idem
  • Revert "idem"
  • Revert "apparently needed"
  • flags autoconf
  • yet another
  • warnings
  • warnings
  • dashes?
  • Be more standard
  • ?
  • autoconf is weird
  • wat
  • modern times
  • i don't understand
  • typos and order
  • ????
  • canonicalization
  • o_O
  • cleanup
  • asound governed by configure
  • These are actually linker flags, not libs
  • Do compiler/arch stuff via configure
  • autoheader
  • sweep through autoconf
  • bracket missing
  • missing \
  • more cleanup
  • gnusource is handled in configure
  • move around
  • fix autoconf typo
  • move around again
  • simpify and fix verstamp
  • silence a warning
  • update configure
  • tiny oss fix
  • Merge remote-tracking branch 'refs/remotes/origin/krono/openbsd' into krono/openbsd
  • fix for oss
  • be early
  • whitespace cleanup
  • :x
  • Merge remote-tracking branch 'origin/Cog' into krono/openbsd
  • execinfo in autoconf
  • improve autoconf and travis
  • improve autoconf, include limits
  • fix oss
  • on NAS
  • gcc
  • on OSS
  • not necessary anymore
  • govern largefile via autoconf
  • gen configure
  • fix use of DEBUGVM macro
  • fix install
  • fix NAS again
  • travis
  • again travis
  • ubuntu, wat
  • travis
  • travis
  • travis
  • now, travis?
  • reset somehow
  • Revert "reset somehow"
  • try again travis
  • fix finding a header
  • some gccs are newer, some are older
  • fix and gen configure
  • govern -z now via autoconf
  • fix and gen configure
  • fix configure
  • use HAVE_EXECINFO_H to identify execinfo
  • fix basic file prims header order
  • tiny fix for uuid
  • promote limits.h to sq.h
  • gen configure (uuid)
  • Merge remote-tracking branch 'origin/Cog' into krono/openbsd
  • adapt new builds

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"The Yakshaving (#226)"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226"}}}</script>
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

akgrant43
 

@j4yk commented on this pull request.


In platforms/unix/plugins/SqueakSSL/sqUnixSSL.c:

> @@ -0,0 +1,18 @@
+/* -*- mode: c; -*- */
+/****************************************************************************
+ *   PROJECT: SqueakSSL implementation for Unix
+ *   FILE:    sqUnixSSL.c
+ *   CONTENT: SSL platform functions
+ *
+ *   AUTHORS: Tobias Pape (topa)
+ *               Hasso Plattner Institute, Postdam, Germany

typo ;-)


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@j4yk commented on #226"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226#pullrequestreview-102894473"}}}</script>
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

akgrant43
In reply to this post by akgrant43
 

@krono commented on this pull request.


In platforms/unix/plugins/SqueakSSL/sqUnixSSL.c:

> @@ -0,0 +1,18 @@
+/* -*- mode: c; -*- */
+/****************************************************************************
+ *   PROJECT: SqueakSSL implementation for Unix
+ *   FILE:    sqUnixSSL.c
+ *   CONTENT: SSL platform functions
+ *
+ *   AUTHORS: Tobias Pape (topa)
+ *               Hasso Plattner Institute, Postdam, Germany

GOD DAMMIT


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@krono commented on #226"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226#discussion_r173668953"}}}</script>
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

akgrant43
In reply to this post by akgrant43
 

@krono pushed 1 commit.


You are receiving this because you are subscribed to this thread.
View it on GitHub or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@krono pushed 1 commit in #226"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226/files/b430377d13f1493c2dc8b1ec13aa32cc526c6f20..63649583e5caf4d543a4ec9e12642ca1329f31f4"}}}</script>
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

akgrant43
In reply to this post by akgrant43
 


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@krono in #226: \u003c!-- yolo --\u003e"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226#issuecomment-373497744"}}}</script>
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] The Yakshaving (#226)

akgrant43
In reply to this post by akgrant43
 

Merged #226.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Merged #226."}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/226#event-1524133211"}}}</script>