[users at bb.net] builedbot try with a patch

[Allen Sanderson]

Hello,

A new user to buildbot try.

Recently I wanted to sent buildbot try a patch —diff=patch.txt

After fixing bug in twisted (https://github.com/buildbot/buildbot/issues/3626) I was able to get the follow error message

2017-11-08 09:37:56-0700 [-] Log opened.
2017-11-08 09:37:56-0700 [-] using 'pb' connect method
2017-11-08 09:37:56-0700 [-] job created
2017-11-08 09:37:56-0700 [-] Starting factory <twisted.spread.pb.PBClientFactory instance at 0x105f76290>
2017-11-08 09:37:56-0700 [Broker,client] Delivering job; comment= None
2017-11-08 09:37:56-0700 [Broker,client] Unhandled Error
	Traceback (most recent call last):
	Failure: sqlalchemy.exc.IntegrityError: (IntegrityError) NOT NULL constraint failed: sourcestamps.repository u'INSERT INTO sourcestamps (branch, revision, patchid, repository, codebase, project, sourcestampsetid) VALUES (?, ?, ?, ?, ?, ?, ?)' (None, None, 71, None, '', '', 3005)
	
2017-11-08 09:37:56-0700 [-] Stopping factory <twisted.spread.pb.PBClientFactory instance at 0x105f76290>
2017-11-08 09:37:56-0700 [-] Main loop terminated.

The error message is less than helpful.

Finally I noticed in the help message for buildbot try —help these following

      --repository=        Repository to use, instead of path to working
                           directory.

 —diff=patch.txt —repository=.

That seemed to work.

As such, it would seem to me that the help message for —diff needs to include details that —repository is also needed or there is a bug in the parsing so a null can be excepted for repository.

Cheers,

Allen


Allen Sanderson
SCI Institute
University of Utah



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.buildbot.net/pipermail/users/attachments/20171108/6bc434af/attachment.html>