Thomas Holmes added the comment:
It's quite alright. Thanks for finishing it up. I realize I ended up a by out
of my depth with regards to packaging's internals and the nuance involved in
this fix. Thanks for all the help.
--
___
Pyth
Thomas Holmes added the comment:
Still working with Éric to determine the proper behavior of
get_reinitialized_commands. Latest patch is known incorrect but a step closer
to our destination.
--
___
Python tracker
<http://bugs.python.
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file23390/1bb1132840e6.diff
___
Python tracker
<http://bugs.python.org/issue12344>
___
___
Python-bugs-list m
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file23388/7db732ac6796.diff
___
Python tracker
<http://bugs.python.org/issue12344>
___
___
Python-bugs-list m
Thomas Holmes added the comment:
I got a bit distracted and I think had some questions on the review. I will go
back over the state of this patch and get some feedback to you tonight or
tomorrow evening.
--
___
Python tracker
<h
Thomas Holmes added the comment:
I mean that msi is _not_ set as a valid format. I will verify this evening.
--
___
Python tracker
<http://bugs.python.org/issue12
Thomas Holmes added the comment:
> > It also creates an exe installer, not an MSI.
> Why would you think it creates an MSI? bdist_wininst is not bdist_msi.
My concern for MSI is that this issue is referencing a change to MSI
generation. I never had any expectation for wininst to ge
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22625/115c0c10b3ba.diff
___
Python tracker
<http://bugs.python.org/issue12344>
___
___
Python-bugs-list m
Changes by Thomas Holmes :
Removed file: http://bugs.python.org/file22624/7c61eba07f3f.diff
___
Python tracker
<http://bugs.python.org/issue12344>
___
___
Python-bug
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22624/7c61eba07f3f.diff
___
Python tracker
<http://bugs.python.org/issue12344>
___
___
Python-bugs-list m
Thomas Holmes added the comment:
I have added a test and made some additional modifications. I had to modify the
very simple MyCmd object in test_command_cmd.py so that it would have some user
options with default values and support finalization
Changes by Thomas Holmes :
--
nosy: +thomas.holmes
___
Python tracker
<http://bugs.python.org/issue12537>
___
___
Python-bugs-list mailing list
Unsubscribe:
Thomas Holmes added the comment:
> See also three comments about style on
> http://bugs.python.org/review/8668/diff2/2845:3011/7845
I'm not sure I follow? The patch I attached does not violate the style
guidelines that you indicate in the comments. The only failing I
Thomas Holmes added the comment:
I was having a separate difficulty with bdist. Apparently -m packaging.run and
pysetup3 run don't perform identically, but that isn't relevant for this bug.
Regarding bdist, I got bdist_wininst to work but I had to modify the code in
place, it did n
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22615/d863f392c094.diff
___
Python tracker
<http://bugs.python.org/issue12344>
___
___
Python-bugs-list m
Thomas Holmes added the comment:
I have made a patch based on your suggestions made to higery, Éric. I have two
questions:
1) In Distributions.get_reinitialized_command should the reinitialization of
the subcommands also get passed the kwargs? Unfortunately my understanding of
the (sub
Thomas Holmes added the comment:
bdist does not appear to be enabled in 3.3 development branch, is this correct?
--
___
Python tracker
<http://bugs.python.org/issue12
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22612/6e15ba060803.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bugs-list m
Thomas Holmes added the comment:
I have made the change you suggested, creating a new list and simply amending
to minimize the diff. This new patch has been attached.
I looked through the rest of database.py and did not see any other generators
that appeared to erroneously hold a file handle
Thomas Holmes added the comment:
Oh and thank you very much for your input. My apologies for the initial 9
e-mail spam when I created the issue, I bumbled the remote HG repository patch
generation :)
--
___
Python tracker
<http://bugs.python.
Thomas Holmes added the comment:
The output in my initial message is the output of the tests with them enabled
but pre database.py patch. Once the patch is applied all packaging tests that
run on my system pass.
I was 50/50 on whether or not to use the internal function and I just sort of
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22591/dcd66ae649b1-2.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bug
Changes by Thomas Holmes :
Removed file: http://bugs.python.org/file22590/dcd66ae649b1.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bug
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22590/dcd66ae649b1.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bugs-list m
Changes by Thomas Holmes :
Removed file: http://bugs.python.org/file22589/dd470b122f32.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bug
Changes by Thomas Holmes :
Added file: http://bugs.python.org/file22589/dd470b122f32.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bugs-list m
Changes by Thomas Holmes :
Removed file: http://bugs.python.org/file22588/f333cd40cd56.diff
___
Python tracker
<http://bugs.python.org/issue12504>
___
___
Python-bug
Changes by Thomas Holmes :
--
keywords: +patch
Added file: http://bugs.python.org/file22588/f333cd40cd56.diff
___
Python tracker
<http://bugs.python.org/issue12
New submission from Thomas Holmes :
Functions test_uninstall.test_uninstall and test_uninstall.test_remove_issue
were disabled for win32. Upon enabling them they generated failures.
I have worked up a patch that refactors a packaging.Distribution function
_get_records from using a generator
Changes by Thomas Holmes :
--
assignee: tarek
components: Distutils2
nosy: alexis, eric.araujo, tarek, thomas.holmes
priority: normal
severity: normal
status: open
title: Uninstall has disabled windows tests
versions: Python 3.3
___
Python tracker
Changes by Thomas Holmes :
--
nosy: +thomas.holmes
___
Python tracker
<http://bugs.python.org/issue5342>
___
___
Python-bugs-list mailing list
Unsubscribe:
Thomas Holmes added the comment:
I setup a proxy and it seems to be working properly. This is on win7 x64
professional using Python 2.7.1
I didn't using any sniffing software but if I broke the proxy the code broke.
When I enabled the proxy it started working again. My proxy log also
Changes by Thomas Holmes :
--
nosy: +thomas.holmes
___
Python tracker
<http://bugs.python.org/issue828450>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Thomas Holmes :
--
nosy: +thomas.holmes
___
Python tracker
<http://bugs.python.org/issue8501>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Thomas Holmes :
--
nosy: +thomas.holmes
___
Python tracker
<http://bugs.python.org/issue12449>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Thomas Holmes :
--
nosy: +Thomas.Holmes
___
Python tracker
<http://bugs.python.org/issue11873>
___
___
Python-bugs-list mailing list
Unsubscribe:
Thomas Holmes added the comment:
I made a new patch off of the 2.7 trunk version. I think I have handled some
of the issues more cleanly.
Please see Py(27)7325.diff
I addressed the issue of using a relative path in the tempdir to achieve the
'guaranteed' ability to write r
Thomas Holmes added the comment:
One other thing that crossed my mind while I was thinking over this today.
Instead of just relative pathing to '.' I should probably change my path to
gettempdir() and then relative path to it since the location of the python
executable may not b
Thomas Holmes added the comment:
I agree, I do not feel like the precise changes to the tests feel completely
ideal. I feel that this problem stems from the fact that the nameCheck
function as originally written doesn't seem to completely serve its originally
intended purpose.
The ori
Changes by Thomas Holmes :
--
nosy: -Thomas.Holmes
___
Python tracker
<http://bugs.python.org/issue1752919>
___
___
Python-bugs-list mailing list
Unsubscribe:
Thomas Holmes added the comment:
Please disregard, I commented on the wrong issue.
--
___
Python tracker
<http://bugs.python.org/issue1752919>
___
___
Python-bug
Thomas Holmes added the comment:
I agree, I do not feel like the precise changes to the tests feel completely
ideal. I feel that this problem stems from the fact that the nameCheck
function as originally written doesn't seem to completely serve its originally
intended purpose.
The ori
Thomas Holmes added the comment:
As a side note, this was done mostly as an exercise for myself and as a
learning experience. Any feedback would be appreciated regardless of any
decision on the status of this bug.
--
___
Python tracker
<h
Thomas Holmes added the comment:
I have created a patch for this for Python 3.1 and included an update to the
unit tests. The tests were never checking for a relative path and if they did
would pas it even when it would have failed due to liberal use of
os.path.abspath()
The test fix is
44 matches
Mail list logo
