Would you please give an example path_mask value here? And add a docstring for the "volume" parameter?

Would you please give an example path_mask value here? And add a docstring for the "volume" parameter?

I'm not really sure about this volume parameter. I'd probably never use it, but it's used by the koji.pathinfo.work function. The first part of your note: done.

Out of curiosity, can you give an example where this would be useful?

Out of curiosity, can you give an example where this would be useful?

It could be used for reusing a previously uploaded data, so we don't need to upload them twice.

Please do not add non-exported methods inside of RootExports. Yes, there may be a few very old ones in there, but we do not need to compound any such errors.

I am opposed to providing this sort of global files api. Koji is not simply a file server. This call provides unstructured views into the file structure. This is too much like having apps browse the filesystem directly (something we discourage).

Can we please start with an issue detailing your current needs that led to this work?

Please do not add non-exported methods inside of RootExports. Yes, there may be a few very old ones in there, but we do not need to compound any such errors.

where should I place it then? Maybe it will be enough to rename it so it starts from "__" (double underscore)?

where should I place it then?

outside of RootExports. Top level function.

However, before you do any more work on this, please file the issue for discussion. There are deeper problems here than the code.

Can we please start with an issue detailing your current needs that led to this work?

https://pagure.io/koji/issue/1207

closing this pending discussion in #1207. If work still needs to be done, we can revisit.