[JIRA] [p4] (JENKINS-24599) Parameters are not resolved in stream

42 views
Skip to first unread message

morne.joubert@u-blox.com (JIRA)

unread,
Sep 5, 2014, 9:38:37 AM9/5/14
to jenkinsc...@googlegroups.com
Issue Type: Bug Bug
Assignee: Unassigned
Components: p4
Created: 05/Sep/14 1:36 PM
Description:

I pass the stream in as a parameter: perforceStream

If i use ${perforceStream} then i get the following:

ERROR: Unable to setup workspace: com.perforce.p4java.exception.RequestException: Error in client specification.
Error detected at line 18.
Stream '${perforceStream}' must begin with '//'.

Project: Jenkins
Priority: Major Major
Reporter: Morne Joubert
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira

daniel@beckweb.net (JIRA)

unread,
Sep 5, 2014, 4:41:33 PM9/5/14
to jenkinsc...@googlegroups.com

Asking for a new feature. Any field is by default not replacing variables like this.

Change By: Daniel Beck (05/Sep/14 8:39 PM)
Issue Type: Bug New Feature

mrose@sjm.com (JIRA)

unread,
Sep 5, 2014, 4:54:33 PM9/5/14
to jenkinsc...@googlegroups.com

Is this really a new feature? Token substitution works fine for the manual client setup option (excluding matrix projects).

pallen@perforce.com (JIRA)

unread,
Oct 6, 2014, 11:22:19 AM10/6/14
to jenkinsc...@googlegroups.com
Paul Allen commented on New Feature JENKINS-24599

A quick workaround would be '//${streamDepot}/${codeline}' or '//${streamPath}

The '//' is a sanity check; to use a ${var} param I would need to test the expanded var. However, depending on how you set the ${var} it may not be valid on the page check.

daniel@beckweb.net (JIRA)

unread,
Oct 6, 2014, 12:31:18 PM10/6/14
to jenkinsc...@googlegroups.com

Michael Rose: My apologies; I'm just doing basic checks on incoming issues and don't know all the plugins – it read like a feature request given the error message, and many users have a habit to report minor feature requests/improvements as critical bugs

Change By: Daniel Beck (06/Oct/14 4:30 PM)
Issue Type: New Feature Bug

mrose@sjm.com (JIRA)

unread,
Oct 6, 2014, 12:52:18 PM10/6/14
to jenkinsc...@googlegroups.com

No problem Daniel Beck and full disclosure I filed a couple ticket last week that ended up not being "defects" or "features" (if you get my drift).

Paul Allen: The workaround looks fine to me. Are $streamDepot, $codeline, $streamPath example variables or real keywords? And if they are real keywords are there any others (e.g. $clientName)?

pallen@perforce.com (JIRA)

unread,
Oct 7, 2014, 5:31:19 AM10/7/14
to jenkinsc...@googlegroups.com
Paul Allen commented on Bug JENKINS-24599

No, they were just example variables.

The only 'special' variables are: ${label}, ${change}, ${status}, ${pass}, ${fail}. They are reserved for the Review/Shelve feature (avoid using them as 'axis' names in a multi config project).

pallen@perforce.com (JIRA)

unread,
Oct 13, 2014, 5:56:20 AM10/13/14
to jenkinsc...@googlegroups.com
Paul Allen resolved Bug JENKINS-24599 as Won't Fix

As outlined the sanity check can't necessarily expand the ${var}.

The alternative would be to remove the check altogether. If common use changes from explicit paths to expanded vars then I may change the sanity check.

Change By: Paul Allen (13/Oct/14 9:55 AM)
Status: Open Resolved
Resolution: Won't Fix

mrose@sjm.com (JIRA)

unread,
Jan 9, 2015, 2:03:55 PM1/9/15
to jenkinsc...@googlegroups.com

I just tried using the '//${streamDepot}/${codeline}' workaround and the build errored out.
The error was

12:50:09 ERROR: Unable to setup workspace: com.perforce.p4java.exception.RequestException: Stream '//${P4DEPOT}/${P4ROJ}' doesn't exist.
12:50:09
12:50:09 ERROR: Unable to setup workspace: com.perforce.p4java.exception.RequestException: Stream '//${P4DEPOT}/${P4ROJ}' doesn't exist.

mrose@sjm.com (JIRA)

unread,
Jan 9, 2015, 2:05:53 PM1/9/15
to jenkinsc...@googlegroups.com
 
Michael Rose edited a comment on Bug JENKINS-24599

I just tried using the '//${streamDepot}/${codeline}' workaround and the build errored out.
The error was

12:50:09 ERROR: Unable to setup workspace: com.perforce.p4java.exception.RequestException: Stream '//${P4DEPOT}/${P4CODELINE}' doesn't exist.
12:50:09
12:50:09 ERROR: Unable to setup workspace: com.perforce.p4java.exception.RequestException: Stream '//${P4DEPOT}/${P4CODELINE}' doesn't exist.

I define P4DEPOT and P4CODELINE using dynamic (but readonly) parameters.

Reply all
Reply to author
Forward
0 new messages