Re: Fedora 34 Change: Deprecate python-mock (Self-Contained change proposal)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Miro Hrončok <mhroncok@xxxxxxxxxx> writes:

> On 08. 02. 21 20:38, Robbie Harwood wrote:
>> Robbie Harwood <rharwood@xxxxxxxxxx> writes:
>> 
>>> Ben Cotton <bcotton@xxxxxxxxxx> writes:
>>>
>>>> A simple `sed` can be applied in `%prep` as a temporary (or even
>>>> permanent) downstream solution.
>>>>
>>>> In most cases, performing the following replacement should be enough:
>>>>
>>>>   s/^(\s*)import mock/\1from unittest import mock/
>>>>   s/^(\s*)from mock import /\1from unittest.mock import /
>>>
>>> a couple lines of sed to all (affected) specfiles.  I hope I have
>>> misunderstood, because that has no mechanism to get the changes back
>>> into upstreams.  Could you clarify what you intend to do?
>> 
>> Turns out this is indeed what they meant.  I would like to reiterate
>> my concern that this has no mechanism to get the changes back into
>> upstreams: it's just Fedora deviating further from the rest of the
>> world, not leading the charge.
>
> Not sure who you mean by "them" in this case,

Change authors.  So, you.

> but doing this downstream only was never my intention. I am the change
> owner.

You have already replied to one of the PRs
https://src.fedoraproject.org/rpms/python-requests-gssapi/pull-request/1
to comment that it couldn't be merged.  It follows the downstream-only
sed approach, you'll note.

Thanks,
--Robbie

Attachment: signature.asc
Description: PGP signature

_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux