On Thu, Dec 12, 2013 at 01:44:34PM +0300, Kanstantsin Shautsou wrote:
I did more debug and found : - .mrev has duplicated entries during creation - https://github.com/openSUSE/open-build-service/issues/529
Yeah, that's a bug in the API or WEBUI.
- no need to increase revision when makeolder isn?t set
Why not? That really depends on your use case. As copying always resets the "rebuild counter", we always bump the revision so that the freshly build packages are "newer" than the old ones.
- why you do renumbering for ?withhistory??
Because we can't have multiple revisions with the same id. If the package did not exist before, it will get the same ids.
- lsrev() and addmeta() are changing hashes for SERVICE entries => it enough to just copy existed .rev with MD5SUMS files
Not in all cases. Maybe if "noservice" is given. Otherwise, all services are re-run (or course only for the latest revision).
- why do you need random generator for SERVICE entries? It can avoid new revision when nothing changed (related https://github.com/openSUSE/open-build-service/issues/278 )
Because the result does not only depend on the package content, but also on the services configured in the project. And, maybe more important, if the service is of the "fetch this url" type, the result may be different files, so we need to have a different identifier for the result. That's not true for package links, here the result only depends on the link content, so we don't need to have to create unique ids.
- copying .rev without changes allow to be in sync with ?history? entries, i think it make sense to pass ?withhistory? option to rep_server so it copy ?history? and meta files?
checkin history and meta files only live on the source server, so passing that to the repo server makes no sense. Cheers, Michael. -- Michael Schroeder mls@suse.de SUSE LINUX Products GmbH, GF Jeff Hawn, HRB 16746 AG Nuernberg main(_){while(_=~getchar())putchar(~_-1/(~(_|32)/13*2-11)*13);} -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-buildservice+owner@opensuse.org