<div dir="ltr"><div><div><div><div><div>Awesome! Thanks for doing this.<br><br></div>Technical comments:<br></div>1) Not sure if we need all those directory rpmdefines. One of the nice things about the CentOS layout is that you can just --define "_topdir $PWD" and the rest of the dirs just fall into place.<br>

<br></div>2) I'm not sure about dead.package check in load_spec. We can't formally adopted that indicator, and if we do, we would probably not put the dead.package file in the SPECS dir. Probably either drop the check entirely, or move it out in front of the loop and check for it in the top directory.<br>

<br></div>3) Splitting in the metadata. I wonder if we shouldn't just use the default whitespace split .split(None, 1) instead of splitting specifically on a single space. It's a little more forgiving if someone tries to manually edit a metadata file. Also removes the need for stripping.<br>

<br></div>4) NotImplemented. You have a number of overrides that raise NotImplementedError. I think perhaps some of these (the non-koji ones) would "just work" if we did not override them (or maybe I'm being overly optimistic).<br>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 14, 2014 at 3:07 AM, Brian Stinson <span dir="ltr"><<a href="mailto:bstinson@ksu.edu" target="_blank">bstinson@ksu.edu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Greetings everyone,<br>
<br>
I spent a little bit of time to see if pyrpkg<br>
(<a href="https://fedorahosted.org/rpkg/" target="_blank">https://fedorahosted.org/rpkg/</a>) could help with the "building RPMS from<br>
a git repo" workflow. Pyrpkg includes methods that can interact with git<br>
(cloning, downloading sources etc), koji, and the local mock/rpmbuild<br>
build processes among other things. You may know pyrpkg better as the<br>
backend for fedpkg in Fedora.<br>
<br>
I have a very bare-bones configuration at<br>
<a href="https://bitbucket.org/bstinsonmhk/centpkg" target="_blank">https://bitbucket.org/bstinsonmhk/centpkg</a> that I thought I would share.<br>
Is the community interested in such a tool? If so I'm happy to keep<br>
working on it, perhaps with the help of others who are interested?<br>
<br>
Cheers!<br>
<span class="HOEnZb"><font color="#888888">Brian<br>
<br>
--<br>
Brian Stinson<br>
<a href="mailto:bstinson@ksu.edu">bstinson@ksu.edu</a> | IRC: bstinson<br>
_______________________________________________<br>
CentOS-devel mailing list<br>
<a href="mailto:CentOS-devel@centos.org">CentOS-devel@centos.org</a><br>
<a href="http://lists.centos.org/mailman/listinfo/centos-devel" target="_blank">http://lists.centos.org/mailman/listinfo/centos-devel</a><br>
</font></span></blockquote></div><br></div>