Issue Details (XML | Word | Printable)

Key: PERF-21
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Keith Brophy
Reporter: Mark Richters
Votes: 2
Watchers: 3
Operations

If you were logged in you would be able to see more operations.
Perforce JIRA plugin

call to p4 with invalid option -t

Created: 07/Dec/04 07:41 AM   Updated: 15/Mar/06 04:23 PM
Component/s: Change Indexing
Affects Version/s: 0.5
Fix Version/s: 1.1.3

Time Tracking:
Not Specified

Environment:
Professional Edition, Version: 3.0.3-#75
Windows
Perforce - The Fast Software Configuration Management System. Copyright 1995, 2004 Perforce Software. All rights reserved. Rev. P4/NTX86/2004.2/68597 (2004/09/03).

Participants: Francisco Candalija, Jeff Turner [Atlassian], Keith Brophy and Mark Richters
Since last comment: 2 years, 25 weeks, 1 day ago
Resolution Date: 15/Mar/06 04:23 PM
Labels:


 Description  « Hide
p4 changes is being called with an option -t (see line 2 in output below), but p4 does not know this option.

INFO: Scheduler JiraQuartzScheduler_$_NON_CLUSTERED started.
P4Process.exec: p4 -s -p localhost:1666 -u jira -c vrpc changes -t -l -s submitted @0,@1000
[Filter: profiling] Using parameter [jira_profile]
[Filter: profiling] defaulting to off [autostart=false]
[Filter: profiling] Turning filter off [jira_profile=off]
P4Process.exec().is: java.io.BufferedInputStream@16aa42e
P4Process.exec().isr: java.io.InputStreamReader@40d611
P4Process.exec().es: java.io.FileInputStream@a632be
P4Process.exec().esr: java.io.InputStreamReader@14d659d
P4Process.exec().os: java.io.BufferedOutputStream@4a3158
P4Process.exec().osw: java.io.OutputStreamWriter@ed8eb5
P4Process: Thread is not done yet.
P4Process: Sleeping...
P4Process: Thread is not done yet.
P4Process: Sleeping...
07.12.2004 14:23:44 com.atlassian.jira.web.dispatcher.JiraServletDispatcher init
WARNUNG: Unable to find 'webwork.multipart.saveDir' property setting. Defaulting to javax.servlet.context.tempdir
07.12.2004 14:23:44 com.atlassian.jira.web.dispatcher.JiraServletDispatcher init
INFO: Action dispatcher initialized
07.12.2004 14:23:44 org.apache.catalina.core.StandardHostDeployer install
INFO: Processing Context configuration file URL file:C:\Prg\jakarta-tomcat-5.0.19\conf\Catalina\localhost\manager.xml
From P4:error: Usage: changes [-i -l -c client -m count -s status -u user] [files...]
P4Process.readLine(): error: Usage: changes [-i -l -c client -m count -s status -u user] [files...]
From P4:error: Invalid option: -t.
P4Process.readLine(): error: Invalid option: -t.
From P4:exit: 1
ERROR: P4 exited with an Error!



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Francisco Candalija added a comment - 03/Jan/05 02:53 AM
We are interested in using this plugin, but we are not able because of this issue. Our P4 server version is P4D/LINUX52X86/2001.1/26850 (2001/10/15) and does not support "-t" option, so the Changes Indexer is not working at all.

Is this issue going to be fixed to support these older versions of P4 ? Is it possible to know when this feature would be implemented ? Thank you


Jeff Turner [Atlassian] added a comment - 04/Jan/05 06:21 AM
Francisco,

Mike is our p4 expert, but it currently overseas and mostly
unavailable. He did mention that the -t option may be unnecessary, as
we do not index by date. We'll need to do some internal testing of
this theory, and get back to you on this issue. Apologies for the
delay in the meanwhile.


ATLASSIAN - http://www.atlassian.com
Expert J2EE Software, Services and Support
-------------------------------------------------------------
JIRA Docs: http://www.atlassian.com/software/jira/docs/latest


Keith Brophy added a comment - 20/Oct/05 01:17 AM
Removed '-t' flag as it is no longer required.

Keith Brophy added a comment - 24/Oct/05 12:00 AM
Need to further investigate this issue - as it appears that the collection of returned changes is sorted based on the time value.

Keith Brophy added a comment - 14/Mar/06 11:51 PM
Removed need for '-t' option and recoded to interpret and display commit info correctly - ordered by change number.