Create Task
Maniphest T15665

Rename repo kdesdk-kioslaves to kdesdk-kio
Closed, ResolvedPublic
Actions

Description

The repo https://invent.kde.org/sdk/kdesdk-kioslaves currently only holds one software component, kio_perldoc.
Once ported to WorkerBase (see MR https://invent.kde.org/sdk/kdesdk-kioslaves/-/merge_requests/2), the repo name no longer matches.

Similar to how repo "zeroconf-ioslave" was renamed to "kio-zeroconf" to adapt to the porting (see T15662) the repo name (and resulting release tarball names) ideally would be changed as well.

Question for the start: what name to pick?

"kio-extras-sdk", to follow the old repo idea to hold multiple things of the sdk domain related to kio?
"kio-perldoc", to match the reality there is just one kio slave (soon worker)?
Something else?

(And yes, there could also be a meta-discussion whether each software component should be in its own separate repo, or all related bundled in big repos, trying not to go into that discussion here :) )

Answer as of time of editing(Mon 11th July): doing the minimal change needed and naming it "kdesdk-kioworkers"

And I propose to rename the repo in time for the upcoming KDE Gear 22.08 release, so any such related renamings are done together and the topic can be checked off.

Related Objects

kossebau created this task.Jul 8 2022, 11:56 AM
kossebau updated the task description. (Show Details)Jul 8 2022, 1:30 PM
ltoscano added a subscriber: ltoscano.Jul 8 2022, 2:00 PM

For the record, the repository used to hold more kio modules, but they were obsoleted until only perldoc is left right now.

kossebau added a comment.Jul 8 2022, 2:24 PM

And I just see that other than nwtwork/kio-extras, in sdk the thumbnailers are also in a separate repo, kdesdk-thumbnailers (which also just holds one thumbnailer these days, for gettext po/pot files).
So "kio-extras-sdk" would be rather misleading then. And if keeping this a multi-component repo, should instead keep the component type in the name -> "kdesdk-kioworkers"?

kossebau added a comment.Jul 10 2022, 12:13 PM

Guess with not many people interested in big discussions or with elaborated plans so far this should then just see the most minimal change needed
And the new name would be: "kdesdk-kioworkers"

Anyone objecting? Or with an elaborated plan? :)

kossebau renamed this task from Rename repo kdesdk-kioslaves to ??? to Rename repo kdesdk-kioslaves to kdesdk-kioworkers.Jul 11 2022, 3:35 PM
kossebau updated the task description. (Show Details)
kossebau added a subscriber: bcooksley.Jul 11 2022, 3:39 PM

Updated the description for the perceived current state of discussion.

Given no-one objected or commented so far, assuming it is mostly a * shrug* from everyone on the matter. And personally think it would be good to have this rename sorted together with kio-zeroconf and for the same KDE Gear cycle, so we are done with any repo-renaming related to KIO's new Worker nomenclature.

So could you please execute the rename on next occasion, @bcooksley ?

ltoscano added a comment.Jul 11 2022, 4:14 PM

Ok, so I do have a last minute objections.
This would be the only reference to "workers" in the repository names.

I would be more in favor of just using the kio prefix or suffix (as we still have audiocd-kio), so either kdesdk-kio or kio-kdesdk.

kossebau added a comment.Jul 11 2022, 4:48 PM
In T15665#278122, @ltoscano wrote:

Ok, so I do have a last minute objections.

Darn :)

This would be the only reference to "workers" in the repository names.

I would be more in favor of just using the kio prefix or suffix (as we still have audiocd-kio), so either kdesdk-kio or kio-kdesdk.

Having kio-extras not being about an "extras:" kio worker also messes up the naming patterns.

Hm. With "kdesdk-thumbnailers" around I would right now then vote for "kdesdk-kio", so there is at least some pattern in stuff related to KIO in the sdk section?

Though is it that bad to have "workers" in the name, when we also have "thumbnailers" in the name?

If only someone would reorganize the repo layouts for us instead... :)

bcooksley added a comment.Jul 12 2022, 11:00 AM

Are we all happy / in favor of kdesdk-kio?

kossebau added a comment.Jul 12 2022, 11:39 AM

+1 (as in, not unhappy)

ltoscano added a comment.Jul 12 2022, 5:27 PM

+1 of course

kossebau renamed this task from Rename repo kdesdk-kioslaves to kdesdk-kioworkers to Rename repo kdesdk-kioslaves to kdesdk-kio.Jul 12 2022, 9:28 PM
bcooksley closed this task as Resolved.Jul 13 2022, 10:41 AM
bcooksley claimed this task.

Repository has now been renamed.