Issue 2550866
Created on 2015-02-04 19:51 by tonich, last changed 2019-10-09 02:40 by rouilj.
msg5204 |
Author: [hidden] (tonich) |
Date: 2015-02-04 19:51 |
|
steps to reproduce:
virtualenv ./venv
. ./venv/bin/activate
pip install --upgrade pip
pip install --upgrade setuptools
hg clone http://hg.code.sf.net/p/roundup/code roundup-code
cd roundup-code
pip install --editable .
result
error: [Errno 2] No such file or directory:
'/home/user/src/roundup-code/roundup-mailgw'
|
msg5205 |
Author: [hidden] (jerrykan) |
Date: 2015-02-05 03:09 |
|
I can confirm that I am seeing a similar issue when using the
-e/--editable option with pip. Invoking pip without the -e option does
not result in an error.
So there seems to be some issue with "development mode"[1] that needs
looking into a bit closer.
[1] https://pythonhosted.org/setuptools/setuptools.html#development-mode
|
msg5206 |
Author: [hidden] (tonich) |
Date: 2015-02-05 19:52 |
|
a worked solution in attachement:
works with "pip install ." and "pip install -e ."
tested with:
pip (6.0.7)
setuptools (12.0.5)
|
msg5207 |
Author: [hidden] (jerrykan) |
Date: 2015-02-06 01:14 |
|
This patch works for me as well.
In regard to the patch the use of the map function makes it non-obvious
what is going on (ie. requires more brain power to figure what is
happening). It may be simpler to make the equivalent changes in the
scriptname() function instead.
|
msg5212 |
Author: [hidden] (jerrykan) |
Date: 2015-02-10 07:10 |
|
I have slightly reworked the patch an push the changes to the repository:
http://hg.code.sf.net/p/roundup/code/rev/cdfb1a3fb56f
Thanks for the original patch tonich
|
msg5427 |
Author: [hidden] (techtonik) |
Date: 2016-01-09 17:21 |
|
This change breaks scripts installation with `setup.py install`. Working
on a proper fix for this.
|
msg5428 |
Author: [hidden] (techtonik) |
Date: 2016-01-09 21:11 |
|
The problem is that "pip install --editable ." converts setup.py from
distutils to setuptools script and ignores build_scripts step.
|
msg5429 |
Author: [hidden] (techtonik) |
Date: 2016-01-09 21:13 |
|
setuptools instead calls its own install_egg_scripts() on "develop", which
just copies the stuff verbatim.
|
msg5430 |
Author: [hidden] (jerrykan) |
Date: 2016-01-10 00:03 |
|
@techtonik also see issue2550898
|
msg5435 |
Author: [hidden] (techtonik) |
Date: 2016-01-10 07:13 |
|
@jerrykan aha, thanks for the pointer. I was about to report the same
results. =)
|
msg5568 |
Author: [hidden] (tonich) |
Date: 2016-05-11 11:54 |
|
Another try
|
msg5601 |
Author: [hidden] (tonich) |
Date: 2016-06-14 07:16 |
|
ping. Please take a look on the second patch. It solves both problems
(this and #2550898).
|
msg5889 |
Author: [hidden] (rouilj) |
Date: 2016-07-31 20:41 |
|
Jerrykan or anatoly, can one of you evaluate the second path
on this issue and see if it solves the problem?
If so I'll get it integrated.
-- rouilj
|
msg5962 |
Author: [hidden] (rouilj) |
Date: 2017-04-23 23:59 |
|
ping John, Anatoly
have either of you evaluated the second patch to see if it fixed the issues?
-- rouilj
|
msg5994 |
Author: [hidden] (rouilj) |
Date: 2017-07-29 01:05 |
|
Re-ping does anybody understand what's happening here and if
the second patch fixes it?
-- rouilj
|
msg6077 |
Author: [hidden] (rouilj) |
Date: 2018-06-01 01:59 |
|
Triple ping JerryKan or Anatoly can either of you validate the patch? I
would like to get it into 1.6.0, but we are late in the process. I don't
want to chance breaking something and I don't understand what's happening
here enough to make an assessment.
|
msg6079 |
Author: [hidden] (jerrykan) |
Date: 2018-06-04 05:04 |
|
Hello John,
Great to hear a v1.6.0 release might be coming soon.
This issue really only affects developers/contributors who are working
on the roundup codebase so I don't really see it as being a blocker for
a v1.6.0 release.
I would suggest that if any patch were to be merged, it might be safer
to do it after the v1.6.0 release given that changes could potentially
cause problems for end-users (see #2550898 for an example of where this
has happened before).
SeeYa,
John.
On 01/06/18 11:59, John Rouillard wrote:
>
> John Rouillard added the comment:
>
> Triple ping JerryKan or Anatoly can either of you validate the patch? I
> would like to get it into 1.6.0, but we are late in the process. I don't
> want to chance breaking something and I don't understand what's happening
> here enough to make an assessment.
>
> ________________________________________________
> Roundup tracker <issues@roundup-tracker.org>
> <http://issues.roundup-tracker.org/issue2550866>
> ________________________________________________
>
|
msg6716 |
Author: [hidden] (rouilj) |
Date: 2019-10-09 02:40 |
|
I am closing this. Developers can run from the tarball without the need to install as we can now run python roundup/scripts/roundup_server with what looks like live reload.
I don't think that existed when anatoly wanted live reload in the demo.py script for his jinja2 work.
There is already a reference to this ticket in issue2550899,
one of the distutils -> setuptools tickets.
The work to make sure the patches are polyglot etc. can be done there.
|
|
Date |
User |
Action |
Args |
2019-10-09 02:40:26 | rouilj | set | status: open -> closed resolution: remind -> abandoned messages:
+ msg6716 |
2019-05-17 00:07:07 | rouilj | set | priority: low |
2018-06-04 05:04:16 | jerrykan | set | messages:
+ msg6079 |
2018-06-01 01:59:10 | rouilj | set | messages:
+ msg6077 |
2017-07-29 01:05:14 | rouilj | set | messages:
+ msg5994 |
2017-04-23 23:59:56 | rouilj | set | messages:
+ msg5962 |
2016-07-31 20:41:35 | rouilj | set | nosy:
+ rouilj messages:
+ msg5889 |
2016-07-31 20:30:28 | rouilj | set | resolution: postponed -> remind |
2016-06-14 07:16:19 | tonich | set | messages:
+ msg5601 |
2016-05-11 11:54:47 | tonich | set | files:
+ roundup-setup-2.patch messages:
+ msg5568 |
2016-01-11 22:01:23 | techtonik | set | resolution: fixed -> postponed |
2016-01-10 07:13:35 | techtonik | set | dependencies:
+ recent setup.py change breaks installation of scripts without setuptools |
2016-01-10 07:13:05 | techtonik | set | messages:
+ msg5435 |
2016-01-10 00:03:55 | jerrykan | set | messages:
+ msg5430 |
2016-01-09 21:13:30 | techtonik | set | messages:
+ msg5429 |
2016-01-09 21:11:48 | techtonik | set | messages:
+ msg5428 |
2016-01-09 17:21:09 | techtonik | set | status: closed -> open nosy:
+ techtonik messages:
+ msg5427 |
2015-02-10 07:10:19 | jerrykan | set | status: new -> closed assignee: jerrykan resolution: fixed messages:
+ msg5212 |
2015-02-06 01:14:33 | jerrykan | set | messages:
+ msg5207 |
2015-02-05 19:52:20 | tonich | set | files:
+ roundup-setup.patch keywords:
+ patch messages:
+ msg5206 |
2015-02-05 03:09:23 | jerrykan | set | nosy:
+ jerrykan messages:
+ msg5205 |
2015-02-04 19:51:55 | tonich | create | |
|