[09:02:34] <gutsytechster> Breaking someone else's PR would mean distributing its commit across multiple branches. Is that what we do? Since, the PR has a lot of changes and commits. How do we keep only specific change?
[09:03:23] <gutsytechster> One way I think is to manually copy the changes and add the PR's author as a co-author? Or do we usually take some other route?
[09:06:56] <gutsytechster> @deveshkusingh^^ I think you have already done some work like this.
[09:14:33] <McSinyx[m]> gutsytechster, could you please provide some context
[09:14:46] <McSinyx[m]> I'm curious on what you're referring to
[09:16:28] <gutsytechster> McSinyx[m]: Ah sorry my bad. I am actually working on https://github.com/pypa/pip/pull/7869. However I have to break this PR in multiple PRs with small changes
[09:41:51] <McSinyx[m]> I think that the manual patches apply seems best
[09:42:58] <McSinyx[m]> but I'm not experienced enough to make my suggestion to have any weight though
[10:03:39] <PSFSlack> <deveshkusingh> One thing I can think of is that you manually pull the PR using `git fetch origin refs/pull/7869/head` and that in your new smaller PR, cherry pick the commits if you can and then apply them to your smaller PR
[10:06:40] <PSFSlack> <deveshkusingh> Perhaps this can help: https://stackoverflow.com/questions/30768148/git-split-pull-request-into-smaller-prs-based-upon-the-new-directories-in-the
[10:07:27] <PSFSlack> <deveshkusingh> And yes, manually copying the changes and adding PR's author as co-author seems to be the easiest way to go, and you can avoid manually pulling the PR too
[10:08:31] <PSFSlack> <deveshkusingh> I have burned my hands in git commands a lot of times, so I would tread with care when using them if you go the cherry-pick/rebase route
[10:29:25] <gutsytechster> @deveshkusingh thanks for the approach!
[10:34:51] <gutsytechster> McSinyx[m]: Regarding this https://github.com/pypa/pip/pull/8266#issuecomment-631373951. Surely I was unaware about purelib and platlib directories. So, as you said if any extension is to be installed, it would go into platlib.
[10:35:40] <gutsytechster> So shouldn't pip only consider platlib, as purelib is not the place for extension modules. And pip would actually install the extensions.
[10:36:09] <McSinyx[m]> I blame my burn out after biking under 37 degrees C, but I don't understand what you mean
[10:37:07] <McSinyx[m]> e.g. pip would be installed under purelib I think
[10:38:21] <gutsytechster> McSinyx[m]: XD. Let me rephrase it. Surely I am most of the times difficult to understand. Now, this is the second experience when someone couldn't understand what I said in last two days.
[10:38:38] <gutsytechster> McSinyx[m]: ok, yeah. pip would go to purelib
[10:39:41] <McSinyx[m]> lol actually my physical burnt out is real, I take like 1 min to parse what my parents said in my native language
[10:41:05] <gutsytechster> So, how do we determine where the package should go. I mean would it then depend on the package?
[10:41:22] <gutsytechster> McSinyx[m]: lmao. It happens. Hope you get alright soon.
[10:43:58] <gutsytechster> But shouldn't this work have already been done? I mean when installing, pip would determine the install location beforehand.
[10:46:05] <gutsytechster> I see that pip further passes `lib_location_guesses` to WorkingSet https://github.com/pypa/pip/blob/803a6f201727a3b64faf976854bac70a1e411150/src/pip/_internal/commands/install.py#L414
[11:01:41] <McSinyx[m]> if i understand the code correctly, that part only check if pkgs are installed correctly
[11:02:19] <McSinyx[m]> so yes, the underlying logics for location determination are deep down the call to install given reqs
[11:05:34] <McSinyx[m]> IIRC some uses the distutils scheme directly (wheel?), some rely on setuptools/distutils logic which does the same thing
[13:03:02] <PSFSlack> <deveshkusingh> gutsytechster: I see that PR #7869 was just now updated. So I guess you should ask there if you can take it over and continue it, or the original PR creator still is available to work on it
[13:03:25] <gutsytechster> @deveshkusingh, yeah just saw that.
[13:03:31] <gutsytechster> I was writing the comment over it.