On Tue, 14 Apr 2020 at 12:05, Vít Ondruch <vondr...@redhat.com> wrote: > > > Dne 14. 04. 20 v 0:13 Ondrej Nosek napsal(a): > > TLDR: Is $SUBJ function reasonable to implement in fedpkg? > > Hi, > > some time ago, fedpkg issue tracker got a request [1] for method, that allows > direct builds. That means without sending srpms via "--srpm" argument. > Currently, user's changes can be built: > > fedpkg scratch-build --srpm > > This way, fedpkg sends srpm file to koji. Without "--srpm", fedpkg sends just > (git) hash id to koji. But fedpkg needs modification to send a right hash id > for user changes. Additionally, koji doesn't allow building hash ids from > forked repos. > > > Even if this was possible, that would also mean that the sources have to be > uploaded into look-a-side cache. Then it very much depend what is the content > of the PR. If I am building Ruby nightly snapshots, I don't think it is fair > to pollute look-a-side cache with them and I am afraid this is not the only > case. I wish we had a way to override the look-a-side cache somehow ....
If I understand correctly, this would have to be done, if sources changed only, right? I.e. if there is a change just in patch or a spec file, the sources could be fetched from the main project. Should there be a possibility to upload sources for a fork that would then be moved to the main project when the pull request is merged? > > > Vít > > > The question to the community. Is reasonable to implement this way of > building scratches? --srpm argument would still work as previously. > > There is a suggested PR for this here: [2]. It is not completed yet. > > Risks: > * approach could confuse users. Users are used to work with fedpkg > differently. > * koji would have to allow these builds > * more code complexity; currently there is a way how to reach your result > even without this function > > Thanks > Ondrej > > [1] https://pagure.io/fedpkg/issue/282 > [2] https://pagure.io/fedpkg/pull-request/390 > > _______________________________________________ > devel mailing list -- devel@lists.fedoraproject.org > To unsubscribe send an email to devel-le...@lists.fedoraproject.org > Fedora Code of Conduct: > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: > https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org > > _______________________________________________ > devel mailing list -- devel@lists.fedoraproject.org > To unsubscribe send an email to devel-le...@lists.fedoraproject.org > Fedora Code of Conduct: > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: > https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org _______________________________________________ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
