Activity

From 2013-07-09 to 2013-08-07

2013-08-07

23:00 Task #50847: get rid of before and after stage step foo
hey Felix, can you enlighten what you think is wrong codewise? Is it just the duplication, the nesting, or? Would be ... Rens Admiraal
10:48 Task #50847: get rid of before and after stage step foo
Hey Christopher,
i think in most cases you will only need one or two dependant tasks and i think it's much nicer (...
Felix Oertel
10:24 Task #50847 (Needs Feedback): get rid of before and after stage step foo
We discussed some different approaches to integrate the new tasks into the workflow (Rens, some others and me).
On...
Christopher Hlubek
11:39 Bug #50557: workflow->removeTask does not work any more and is destroying the array structure
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22698
Gerrit Code Review
11:30 Bug #49973 (Resolved): add version to composer.json
Applied in changeset commit:f9767de98c6cdb23c9b14582175445b5184573f1. Anonymous
11:24 Task #50854 (Under Review): cleanup options in node
Karsten Dambekalns
03:15 Task #50854: cleanup options in node
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22903
Gerrit Code Review
03:14 Task #50854 (Resolved): cleanup options in node
@$port@ and @$username@ are options, while @$hostname@ and @$name@ are properties ... why?
let's get that straight ;)
Felix Oertel
02:51 Task #50853: Move createDefaultLogger to factory
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22902
Gerrit Code Review
02:49 Task #50853 (Resolved): Move createDefaultLogger to factory
come on ;) @createDefaultLogger()@ does really not belong into the command controller ...
let's move it
Felix Oertel
02:30 Task #50852: Make Logger available as soon as possible
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22901
Gerrit Code Review
02:28 Task #50852: Make Logger available as soon as possible
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22901
Gerrit Code Review
02:23 Task #50852 (Rejected): Make Logger available as soon as possible
We should make the logger available to the deployment, as soon as possible, espacially before @require($deploymentPat... Felix Oertel

2013-08-06

23:12 Task #50847 (Rejected): get rid of before and after stage step foo
hey,
after the recent changes, introducing packaged deployment (which i love!), we ended up with some new, kind of...
Felix Oertel
15:50 Feature #50151: Add fetchAllTags option to GitCheckoutTask
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22848
Gerrit Code Review
15:44 Bug #49973: add version to composer.json
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22247
Gerrit Code Review
15:44 Revision f9767de9: [FEATURE] remove version from composer.json
Current composer.phar expects version to not be empty.
Fixes: #49973
Change-Id: Ice288779fb9eda06f569de44d2efa11bdb0...
Felix Oertel
13:22 Feature #49489: Add noRecursiveSubmodules option to GitCheckoutTask
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22396
Gerrit Code Review
12:30 Bug #50779 (Resolved): Handling of branch option in GitCheckout Task does not work as expected.
Applied in changeset commit:daf9ad4eda93fbf24ec16a8c2252f56191a246f5. Anonymous
11:44 Bug #50779: Handling of branch option in GitCheckout Task does not work as expected.
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22837
Gerrit Code Review
11:21 Bug #50779: Handling of branch option in GitCheckout Task does not work as expected.
I found the reason for the problem. ... Martin Ficzel
11:44 Revision daf9ad4e: [BUGFIX] Fixed a misleading deprecation message and wrong option mapping
The options used in setDeprecatedTaskOptions were misspelled in lowercase where lowerCamelCase is needed. This patch ... Martin Ficzel

2013-08-05

10:01 Bug #50554: CopyConfigurationTask does not use the node-option "password"
The problem can be solved in a better way with the exchanging of ssh-keys. Since there is no good way to pass a passw... Martin Ficzel
09:59 Bug #49973: add version to composer.json
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22247
Gerrit Code Review
09:56 Bug #50779: Handling of branch option in GitCheckout Task does not work as expected.
To me it looks like an incomplete merge. Is it possible that a change in the option handling was not included in the ... Martin Ficzel
09:55 Bug #50779 (Resolved): Handling of branch option in GitCheckout Task does not work as expected.
Since the last updates the prevoiusly used option "git-checkout-branch" is replaced by "typo3.surf:gitcheckout[branch... Martin Ficzel

2013-08-02

20:42 Bug #48135: funtion prepareCommand() in class ShellCommandService
ready for review
https://review.typo3.org/22776
Dietrich Heise
20:41 Bug #48135: funtion prepareCommand() in class ShellCommandService
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22776
Gerrit Code Review

2013-08-01

17:05 Bug #48135 (Needs Feedback): funtion prepareCommand() in class ShellCommandService
Could you please make a change in gerrit for this? Tobias Liebig
17:00 Bug #45072 (Closed): Possible error with git submodules
Tobias Liebig
16:54 Bug #41893 (Needs Feedback): Could not retrieve sha1 of git branch master
Like Cedric, i think this is an issue in your setup, not in Surf itself.
Surf expects the remote node has access t...
Tobias Liebig
16:50 Feature #37077 (Needs Feedback): Surf should also support local checkout and rsync to server
Hej Thomas,
i think this issue is solved by the new "flexible transfer method", which we merged recently.
https:/...
Tobias Liebig
16:46 Task #50664 (New): check for unresolved TODOs in the code
Tobias Liebig
16:45 Feature #50663 (New): support scp transfer
* Implement a Transfer\ScpTask (like the RsyncTask)
* enable it in BaseApplication::registerTasksForTransferMethod
Tobias Liebig
16:44 Feature #50662 (New): support ftp transfer
* Implement a Transfer\FTPTask (like the RsyncTask)
* enable it in BaseApplication::registerTasksForTransferMethod
Tobias Liebig
16:41 Bug #50661 (New): documentation
* Create documentation in reST
* Render documentation via readthedocs.org
* Publish on docs.typo3.org if possible
...
Tobias Liebig
16:41 Task #50660 (New): concept for product website
Create a concept for a TYPO3 Surf product website and plan/coordinate the implementation.
* Target groups
* te...
Tobias Liebig
16:38 Feature #50658 (Accepted): refactor ShellCommandService to have separated access to StdIn StdErr and StdOut
Tobias Liebig
16:38 Feature #50658: refactor ShellCommandService to have separated access to StdIn StdErr and StdOut
great to see someone is willing to work on this.
Go for it. Looking forward to your change in gerrit.
Tobias Liebig
16:37 Feature #50658: refactor ShellCommandService to have separated access to StdIn StdErr and StdOut
me me me ... Felix Oertel
16:35 Feature #50658 (Resolved): refactor ShellCommandService to have separated access to StdIn StdErr and StdOut
make use of the symfony/process component to separate the stdErr and stdOut streams in the ShellCommandService Tobias Liebig
16:36 Task #50659 (New): Unit tests
check for and implement a bunch of meaningful unit tests Tobias Liebig
16:34 Feature #50657 (Resolved): rsync transfer method: add option the --exclude argument
add an exclude option for rsync task (--exclude arguments) Tobias Liebig
16:32 Task #50656 (New): cleanup task names
refactor names of the tasks (move them into meaningful folders).
define an alias for the “old” names and deprecate t...
Tobias Liebig
16:32 Task #50655 (New): check all shell->excute for correctly escaped arguments
check all shell->excute for correctly escaped arguments Tobias Liebig
16:31 Task #50654 (Resolved): remove setDeprecatedTaskOptions from BaseApplication
remove setDeprecatedTaskOptions from BaseApplication Tobias Liebig
16:31 Bug #50653 (New): rsync transfer method writes empty *REVISION on remote node
write correct *REVISION files on remote when using the rsync transfer method Tobias Liebig
16:30 Feature #50652 (New): ShellCommandService: method to expand paths in commands
add a method in ShellCommandService to replace paths in command
something like:
$command = $this->shell->ex...
Tobias Liebig
16:28 Task #50651 (New): Add @api annotations
check the code for missing @api annotations Tobias Liebig
16:28 Task #50650 (New): Provide better error messages
Provide better error messages for common problems and add some tasks for testing the setup (e.g. SSH connection test) Tobias Liebig
16:27 Feature #50649 (New): command to kickstart a new configuration
Kickstart command for new deployment (keep it simple for 1.0)
* ask some Questions
* Hostname of the target node:...
Tobias Liebig
16:22 Task #50648 (Accepted): enforce usage of “->forStage” instead of “->addTask”
Make sure, we are using "->forStage" instead of the older "->addTask" method Tobias Liebig
16:22 Task #50647 (New): refactor addTask method argument order
To streamline the API, the order of the method arguments in ->addTask should be changed.
First argument should be th...
Tobias Liebig
15:58 Task #50646 (New): adding tasks: make use of afterStage instead of afterTask
Instead of creating unnecessary dependencies between tasks in one stage, we should internally make use of the new ->a... Tobias Liebig
15:55 Feature #50644 (New): helper setter for common options
Currently some options need to be set using the generic setOption setter. For the API it would be nice to have explic... Tobias Liebig
15:54 Feature #50642 (New): Create new ComposerApplication
FlowApplications are deployed using composer and all composer related stuff is implemented in the FlowApplication. Ot... Tobias Liebig
15:48 Feature #50641 (New): Make simple deployment description as slim as possible
A simple deployment description should by as slim as possible.
* Check level of option assignment (deployment, nod...
Tobias Liebig
15:43 Feature #50640 (New): refactor DeploymentService > DeploymentRepository
To support other configuration formats, the DeploymentService should be refactored.
Other formats (instead of the ...
Tobias Liebig
15:32 Feature #50639 (New): support SVN
Like Surf supports Git for as VCS, Surf should support SVN too.
Thus we need a SVNCheckoutTask like the current Gi...
Tobias Liebig

2013-07-30

13:33 Bug #50557: workflow->removeTask does not work any more and is destroying the array structure
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22698
Gerrit Code Review
13:14 Bug #50557 (Resolved): workflow->removeTask does not work any more and is destroying the array structure
Since commit 2b7d957eaf77acddef4028cd965822545b2853b0 the removeTask methods does destroy the structure of its intern... Martin Ficzel
12:09 Bug #50554 (Closed): CopyConfigurationTask does not use the node-option "password"
The CopyConfigurationTask uses the configured username but not the password for copying the files to the node via scp... Martin Ficzel

2013-07-18

13:22 Feature #50151 (Resolved): Add fetchAllTags option to GitCheckoutTask
Without further options, "git fetch" only fetches tags that point at objects reachable from the branch heads (man 1 g... Patrick Pussar
13:07 Feature #49489: Add noRecursiveSubmodules option to GitCheckoutTask
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22396
Gerrit Code Review

2013-07-17

17:43 Revision 17f84d87: [FEATURE] Support flexible transfer and update methods
This change implements new stages and options for a declarative
configuration of the package / transfer / update proc...
Tobias Liebig

2013-07-12

13:29 Bug #49973: add version to composer.json
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/22247
Gerrit Code Review
13:28 Bug #49973 (Resolved): add version to composer.json
Current composer.phar expects version to be set. Felix Oertel

2013-07-09

08:07 Bug #43366: Misleading error message if "expect" is not available
Sure, but then maybe the output of surf could be "optimized" in the following way:
Either
* don't make it bold an...
Mario Rimann
 

Also available in: Atom