onkelbeh / HomeAssistantRepository

:house_with_garden: Home Assistant on Gentoo Linux.
https://git.edevau.net/onkelbeh/HomeAssistantRepository
GNU General Public License v3.0
39 stars 13 forks source link

Update protobuf-python-4.24.3-r1.ebuild #272

Closed EmilienMottet closed 9 months ago

EmilienMottet commented 10 months ago

fix ebuild = in slot

I have this error and I use this fix

 !!! All ebuilds that could satisfy ">=dev-python/protobuf-python-4.24.3[python_targets_python3_11(-)?]" have been masked.
!!! One of the following masked packages is required to complete your request:
- dev-python/protobuf-python-9999::gentoo (masked by: missing keyword)
- dev-python/protobuf-python-4.24.3-r1::HomeAssistantRepository (masked by: invalid: BDEPEND: Improper context for slot-operator "built" atom syntax: dev-libs/protobuf:0/24.3.0=, invalid: RDEPEND: Improper context for slot-operator "built" atom syntax: dev-libs/protobuf:0/24.3.0=)
onkelbeh commented 10 months ago

@inode64 mind taking a look? I don't know exactly what the "=" does in Python, and it was added by you.

inode64 commented 10 months ago

An = is added at the end in the dependencies when the slot in the library changes and you need to compile that program so as not to lose the dependencies. Slot is used to identify when the API changes in a library and thus all applications that depend on it are automatically recompiled.

See: https://devmanual.gentoo.org/general-concepts/dependencies/#slot-dependencies

onkelbeh commented 10 months ago

I get the same error compiling the -r1 version:

inode64 commented 10 months ago

Reviewing the topic, in the end, adding = is redundant and can be removed in this case, since if we block only one slot when we change slots in dev-libs/protobuf we have to have the same one in dev-python/protobuf-python. With which the compilation is forced. Emerge does not allow you to have slot 0/24.3.0 because it considers that it is a specific and unique version.

You can also remove BDEPEND= because pkgcheck shows this warning:

 EmptyGlobalAssignment: version 4.24.3-r1: line 41: empty global assignment: BDEPEND="
onkelbeh commented 9 months ago

Thanks.