[Issue 2105] New - Perforce does not seem to build up links to p4web correctly

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Issue 2105] New - Perforce does not seem to build up links to p4web correctly

frankt-3
https://hudson.dev.java.net/issues/show_bug.cgi?id=2105
                 Issue #|2105
                 Summary|Perforce does not seem to build up links to p4web corr
                        |ectly
               Component|hudson
                 Version|current
                Platform|All
              OS/Version|All
                     URL|
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|perforce
             Assigned to|digerata
             Reported by|frankt






------- Additional comments from [hidden email] Tue Jul 22 17:20:07 +0000 2008 -------
On a perforce project, go to the recent changes link on the job page:
http://<hudson_server>/job/<project>/changes
Click on the changeset number from the summary of changes list:
http://<hudson_server>/job/<project>/<build_number>/changes#detail0
The links to p4web on this page, seem to be built incorrectly:

Example of "Affected Files" link:
http://<perforce_server>/@md=d&cd=//ypost/isd/&c=Bv5@//ypost/isd/main/@md=d&rc=s&rt=s&thb=y&c=ihy@//ypost/isd/main/re_client_srp4/conServer.pl?ac=22

Clicking on the above link produces the following message:
Unintelligible revision specification
'md=d&rc=s&rt=s&thb=y&c=ihy@//ypost/isd/main/re_client_srp4/conServer.pl'.

You can replace the above link with the following link, you get the right thing:
http://<perforce_server>//ypost/isd/main/re_client_srp4/conServer.pl?ac=22


Example of Revision link:
http://<perforce_server>/@md=d&cd=//ypost/isd/&c=Bv5@//ypost/isd/main/@md=d&rc=s&rt=s&thb=y&c=ihy@//ypost/isd/main/re_client_srp4
/conServer.pl?ac=19&&rev1=14&rev2=15

Clicking on the above Revision link give you this message:
Unintelligible revision specification
'md=d&rc=s&rt=s&thb=y&c=ihy@//ypost/isd/main/re_client_srp4/conServer.pl'.

Replacing the above link with this working link, gives you the right thing:
http://<perforce_server>//ypost/isd/main/re_client_srp4/conServer.pl?ac=19&&rev1=14&rev2=15

There is a discussion of the interface to p4web in the O'Reilly book
Practical Perforce by Laura Wingerd on page 156.  It basically says, that you
don't need the stuff between the @ signs when building up URLs for p4web.

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Issue 2105] Perforce does not seem to build up links to p4web correctly

digerata-2
https://hudson.dev.java.net/issues/show_bug.cgi?id=2105



User digerata changed the following:

                What    |Old value                 |New value
================================================================================
                  Status|NEW                       |STARTED
--------------------------------------------------------------------------------




------- Additional comments from [hidden email] Tue Jul 22 17:41:54 +0000 2008 -------
That's interesting.  I'm curious about which version of P4Web you are using.

I have no problem yanking out all of the junk.  I'll make sure this gets into
the next release.

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Issue 2105] Perforce does not seem to build up links to p4web correctly

frankt-3
In reply to this post by frankt-3
https://hudson.dev.java.net/issues/show_bug.cgi?id=2105






------- Additional comments from [hidden email] Tue Jul 22 17:48:06 +0000 2008 -------
I am using:
  P4Web version: P4Web/2007.3/FREEBSD60X86/143793

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]