Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

integrate gsApplicationTools and GsDevKit/zinc #2

Closed
dalehenrich opened this issue Nov 26, 2014 · 3 comments
Closed

integrate gsApplicationTools and GsDevKit/zinc #2

dalehenrich opened this issue Nov 26, 2014 · 3 comments

Comments

@dalehenrich
Copy link
Member

pull out pieces that should be part of zinc project and create zinc http-server and web-socket-server support scripts for zinc ... next part of Zinc Issue 58
dalehenrich added a commit that referenced this issue Nov 26, 2014
@dalehenrich
Issue #2: move ZnServerStarter to the Zinc project
93e4420
dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2:
edbb163 
- rename Application-Server-Tools to GsApplicationTools-Server and do a bit of restructuring
- add test package and test stub
dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2: correct mistakes in baseline for package renames
67fdf02
dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2: load CI group to pick up tests
b704318
dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2:
5b111f7 
Taking approach where instead of subclassing GemServerStarter, subclass GemServer ... then we can do direct customization of the various bits and pieces that need customization on run ... and pass any interesting state via the script call ... script location etc. can be specified by the subclass ...

Added tests for validating the basic api
@dalehenrich
Copy link
Member Author

Going towards a GemServer-centric desing ... it seems that if we allow customization of the GemServer subclass on a per server basis, we can really shrink down the size of the script needed to launch the gem ... port and Gem name are all that are needed to distinguish a particular executable ... perhaps gem-status can be shared somehow as well ... lots of food for thought ... anyway I will make a pass at the GemServer-centric approach and see if things can be adapted for maintenance vm, fastCGI seaside, zinc seaside, zinc webserver and service vms ...

dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2: incorporated the "server boilerplate" Smalltalk code from s…
4dbe1c7 
…tartSeaside30_Adaptor ... push a bunch of stuff around ...
dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2: remove the GemServerStarter classes .... refactor start log…
c5c8812 
…ic to send startGems, restartGems and stopGems directory to the GemServer instance ... return GemServer instance when created, so that inline start and stop logic can be done without requiring lookup...
dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2: get rid of GemServer portInstance variable ... not really n…
4c4cfa8 
…ecessary
@dalehenrich
Copy link
Member Author

  • I think that the *GemOnPort: messages in GemServerRegistry are not implemented correctly, since the set of gems is stop/start/restart when one of the ports matches ... not quite what I think is expected ... just dump them until we have a real use case ...

dalehenrich added a commit that referenced this issue Nov 27, 2014
@dalehenrich
Issue #2: get tests passing?
700ca65
dalehenrich added a commit that referenced this issue Nov 29, 2014
@dalehenrich
Issue #2: continue move towards GemServer centric design ... with Gem…
cd2c3d0 
…Server class>>register:for:on: as the center piece
dalehenrich added a commit that referenced this issue Nov 29, 2014
@dalehenrich
Issue #2: get ready to run scripts ... add test
23e8f5d
dalehenrich added a commit that referenced this issue Nov 29, 2014
@dalehenrich
issue #2: [ci skip] checkpoint .. preparing for first script tests
faedb80
dalehenrich added a commit that referenced this issue Nov 29, 2014
@dalehenrich
Issue #2: start/stop/restart appears to be working but I don't like t…
4ecb59f 
…he level of feedback ... need status command that looks at pid and verifies that pid is there and perhaps some other status information...
dalehenrich added a commit that referenced this issue Nov 29, 2014
@dalehenrich
Issue #2: adjustments for testing
a88c93b
dalehenrich added a commit that referenced this issue Nov 29, 2014
@dalehenrich
Issue #2: add remote vm test
8753501
dalehenrich added a commit that referenced this issue Nov 30, 2014
@dalehenrich
Issue #2: get the GemServerRemoteVMTests>>testStart test to function
6f579af
dalehenrich added a commit that referenced this issue Nov 30, 2014
@dalehenrich
Issue #2: fix mistake in test
774d5e5
dalehenrich added a commit that referenced this issue Nov 30, 2014
@dalehenrich
Issue #2: split out GemServer>>performOnServer: into separate 2.4 and…
35d1bb7 
… 3.x packages ... add new packages and update baseline
dalehenrich added a commit that referenced this issue Nov 30, 2014
@dalehenrich
Issue #2: GemServer>>enableRemoteBreakpointHandling is platform-speci…
45ed3fd 
…fic as well
dalehenrich added a commit that referenced this issue Nov 30, 2014
@dalehenrich
Issue #2: 2.4 version of GemServer>>enableRemoteBreakpointHandling
eb3594e
@dalehenrich
Copy link
Member Author

clean sheet for tests that launch test gemserver

dalehenrich added a commit to GsDevKit/zinc that referenced this issue Nov 30, 2014
@dalehenrich
Issue #58: start using ZnGemServer based on work done for GsDevKit/gs…
785e1cc 
…ApplicationTools#2
dalehenrich added a commit that referenced this issue Nov 30, 2014
@dalehenrich
Issue #2: change GemServer>>defaultScriptBinDirPath to directly acces…
31c1fd7 
…s GemServer class, so that subclasses in different projects can still find the script
HeSe added a commit that referenced this issue Dec 10, 2014
@HeSe
Merge pull request #2 from GsDevKit/master
775e702 
I thin those changes are great! Thank you Dale. I'll try to merge my changes into it later on. Let's keep this as the new master.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dalehenrich