[Buildbot-devel] Fix for wrong user in p4poller

[François Beerten]

Hi all,

There is a bug in p4poller.py when it fetches the descriptions of the changelists. It doesn't set the P4USER environment variable. 

In that situation, Perforce guesses the user which can be wrong. It can also lead to "can't create a new user" errors from the Perforce server when all your licences are used.

Here is the fix:

+++ bb/changes/p4poller.py      2006-04-13 17:55:12.000000000 +0200
@@ -93,6 +93,7 @@
     def _get_change(self, change):
         args = ['describe', '-s', change['num']]
         env = {'P4PORT' : self.p4port,
+               'P4USER' : self.p4user,
                'P4CLIENT' : self.p4client}
         d = getProcessOutput(self.p4bin, args, env)
         d.addCallback(self._process_change, change)




FB. 
--------------------------------------------------------
The information contained in this message or any of its attachments may be privileged and confidential and intended for the exclusive use of the addressee. If you are not the addressee any disclosure, reproduction, distribution or other dissemination or use of this communication is strictly prohibited. If you have received this transmission by error please notify the sender immediately and then delete this email.  EURESYS shall not be liable for any loss of or damage to revenues, profits, goodwill, data, information systems or other special, incidental, indirect, consequential or punitive damages of any kind arising in connection with the use of information contained in this mail or its attachments.  Email transmission cannot be guaranteed to be secure or error free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses.  The sender therefore is in no way liable for any errors or omissions in the content of this message, which may arise as a result of email transmission. If verification is required, please request a hard copy.  Please note that neither EURESYS, nor the sender accepts any responsibility for viruses and it is your responsibility to scan attachments (if any).

--------------------------------------------------------