repo upload similarity

23 views
Skip to first unread message

Lutz Justen

unread,
Oct 11, 2017, 4:41:44 AM10/11/17
to Chromium OS dev
Does repo upload support a way to specify similarity for detecting moves vs delete/adds?
I see that git cl upload has a --similarity option, but what about repo upload?
I have a CL where my local git detected a move, but Gerrit didn't.

Thanks!

Mike Frysinger

unread,
Oct 12, 2017, 12:53:26 AM10/12/17
to Lutz Justen, Chromium OS dev
repo upload is dumb (as in KISS).  it just pushes commits to gerrit.  any checks run on the code are entirely on our hands via the repohooks code, and we don't have anything in there afaik.
-mike

--
--
Chromium OS Developers mailing list: chromiu...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-os-dev?hl=en


Lutz Justen

unread,
Oct 12, 2017, 2:24:51 AM10/12/17
to Mike Frysinger, Chromium OS dev
Got it, thanks! Are we setting a higher threshold than 50%? My changes had similarity between 51% and 54%.

I ended up working around it by doing the move in a separate CL first. 

--

Dr. Lutz Justen
Software Engineer
lju...@google.com

Google Germany GmbH
ABC-Str. 19
20354 Hamburg

Geschäftsführer: Paul Manicle, Halimah DeLaine Prado

Registergericht und -nummer: Hamburg, HRB 86891

Diese E-Mail ist vertraulich. Falls sie diese fälschlicherweise erhalten haben sollten, leiten Sie diese bitte nicht an jemand anderes weiter, löschen Sie alle Kopien und Anhänge davon und lassen Sie mich bitte wissen, dass die E-Mail an die falsche Person gesendet wurde.    

This e-mail is confidential. If you received this communication by mistake, please don't forward it to anyone else, please erase all copies and attachments, and please let me know that it has gone to the wrong person.

Der Inhalt dieser E-Mail spiegelt den derzeitigen Stand der Verhandlungen wider und dient als Basis für weitere Gespräche. Er soll keine rechtlich verbindliche Verpflichtung begründen. Eine solche Verpflichtung wird allein durch einen zwischen allen beteiligten Parteien abgeschlossenen, schriftlichen Vertrag geschaffen.

The above terms reflect a potential business arrangement, are provided solely as a basis for further discussion, and are not intended to be and do not constitute a legally binding obligation. No legally binding obligations will be created, implied, or inferred until an agreement in final form is executed in writing by all parties involved.

Mike Frysinger

unread,
Oct 12, 2017, 11:47:28 AM10/12/17
to Lutz Justen, Chromium OS dev
i have no idea what gerrit settings are tuned to.  there's an internal gerritcodereview-users@ group you could prob inquiry on :).
-mike

Lutz Justen

unread,
Oct 12, 2017, 12:33:40 PM10/12/17
to Mike Frysinger, Chromium OS dev
Thanks. For future reference, it's 60% in Gerrit, whereas git uses 50% by default.
Reply all
Reply to author
Forward
This conversation is locked
You cannot reply and perform actions on locked conversations.
0 new messages