cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-6909) shellCmd and shellRunParam in master.cfg use values for master, fails on Windows slaves if OS X master
Date Tue, 10 Jun 2014 09:39:02 GMT

    [ https://issues.apache.org/jira/browse/CB-6909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14026271#comment-14026271
] 

ASF GitHub Bot commented on CB-6909:
------------------------------------

GitHub user vladimir-kotikov opened a pull request:

    https://github.com/apache/cordova-medic/pull/11

    CB-6909 shellCmd and shellRunParam in master.cfg use values for master, fails on Windows
slaves if OS X master

    shellCmd and shellRunCmd defined in master.cfg depends on the OS of the master, so if
you have a OS X master and Windows client (or vice versa) a lot of steps will fail.
    For some steps, like in wp8 and windows8 sections, we know they'll always run on Windows
so we could always use 'cmd'. But in common sections or for a platform like android, it needs
to work on both.
    
    Fix for [CB-6909](https://issues.apache.org/jira/browse/CB-6909)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/MSOpenTech/cordova-medic CB-6909

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-medic/pull/11.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #11
    
----
commit 8e843e92fee168986859cc34c3adc2302b62e72e
Author: Vladimir Kotikov <v-vlkoti@microsoft.com>
Date:   2014-06-09T13:35:12Z

    Removes platform-dependent shellCmd and shellRunParam from master.cfg

----


> shellCmd and shellRunParam in master.cfg use values for master, fails on Windows slaves
if OS X master
> ------------------------------------------------------------------------------------------------------
>
>                 Key: CB-6909
>                 URL: https://issues.apache.org/jira/browse/CB-6909
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Medic, Windows 8, WP8
>         Environment: OS X master, buildbot 0.8.8, Windows Server 2012R2 slave, buildslave
0.8.8
>            Reporter: Vladimir Kotikov
>            Assignee: Jesse MacFadyen
>              Labels: medic, windows8, wp8
>
> shellCmd and shellRunCmd are defined in master.cfg and used for a lot of the build tasks.
But the value depends on the OS of the master, so if you have a OS X master and Windows client
(or vice versa) a lot of steps will fail.
> {noformat}
> shellCmd="/bin/sh"
> shellRunParam="-c"
> if is_Windows :
>     shellCmd="cmd"
>     shellRunParam="/c"
> {noformat}
> For some steps, like in wp8 and windows8 sections, we know they'll always run on Windows
so we could always use 'cmd'. But in common sections or for a platform like android, it needs
to work on both.
> -----
> It seems that ShellCommand step already uses command interpreter according to slave platform
(see http://docs.buildbot.net/current/manual/cfg-buildsteps.html#step-ShellCommand). So it's
safe to remove this and specify commands for ShellCommand steps just with list of strings.
> Another possible issue is different path delimiter on windows and other platforms. But
in most cases slash instead of backslash works fine on windows. So it is possible to replace
'\\' to '/' in windows build steps.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message