pycryptopp-related hang of unit tests on platforms using buggy Gnu as 2.20 (e.g. MinGW 5.1.x, Ubuntu Karmic) #853
Labels
No labels
c/code
c/code-dirnodes
c/code-encoding
c/code-frontend
c/code-frontend-cli
c/code-frontend-ftp-sftp
c/code-frontend-magic-folder
c/code-frontend-web
c/code-mutable
c/code-network
c/code-nodeadmin
c/code-peerselection
c/code-storage
c/contrib
c/dev-infrastructure
c/docs
c/operational
c/packaging
c/unknown
c/website
kw:2pc
kw:410
kw:9p
kw:ActivePerl
kw:AttributeError
kw:DataUnavailable
kw:DeadReferenceError
kw:DoS
kw:FileZilla
kw:GetLastError
kw:IFinishableConsumer
kw:K
kw:LeastAuthority
kw:Makefile
kw:RIStorageServer
kw:StringIO
kw:UncoordinatedWriteError
kw:about
kw:access
kw:access-control
kw:accessibility
kw:accounting
kw:accounting-crawler
kw:add-only
kw:aes
kw:aesthetics
kw:alias
kw:aliases
kw:aliens
kw:allmydata
kw:amazon
kw:ambient
kw:annotations
kw:anonymity
kw:anonymous
kw:anti-censorship
kw:api_auth_token
kw:appearance
kw:appname
kw:apport
kw:archive
kw:archlinux
kw:argparse
kw:arm
kw:assertion
kw:attachment
kw:auth
kw:authentication
kw:automation
kw:avahi
kw:availability
kw:aws
kw:azure
kw:backend
kw:backoff
kw:backup
kw:backupdb
kw:backward-compatibility
kw:bandwidth
kw:basedir
kw:bayes
kw:bbfreeze
kw:beta
kw:binaries
kw:binutils
kw:bitcoin
kw:bitrot
kw:blacklist
kw:blocker
kw:blocks-cloud-deployment
kw:blocks-cloud-merge
kw:blocks-magic-folder-merge
kw:blocks-merge
kw:blocks-raic
kw:blocks-release
kw:blog
kw:bom
kw:bonjour
kw:branch
kw:branding
kw:breadcrumbs
kw:brians-opinion-needed
kw:browser
kw:bsd
kw:build
kw:build-helpers
kw:buildbot
kw:builders
kw:buildslave
kw:buildslaves
kw:cache
kw:cap
kw:capleak
kw:captcha
kw:cast
kw:centos
kw:cffi
kw:chacha
kw:charset
kw:check
kw:checker
kw:chroot
kw:ci
kw:clean
kw:cleanup
kw:cli
kw:cloud
kw:cloud-backend
kw:cmdline
kw:code
kw:code-checks
kw:coding-standards
kw:coding-tools
kw:coding_tools
kw:collection
kw:compatibility
kw:completion
kw:compression
kw:confidentiality
kw:config
kw:configuration
kw:configuration.txt
kw:conflict
kw:connection
kw:connectivity
kw:consistency
kw:content
kw:control
kw:control.furl
kw:convergence
kw:coordination
kw:copyright
kw:corruption
kw:cors
kw:cost
kw:coverage
kw:coveralls
kw:coveralls.io
kw:cpu-watcher
kw:cpyext
kw:crash
kw:crawler
kw:crawlers
kw:create-container
kw:cruft
kw:crypto
kw:cryptography
kw:cryptography-lib
kw:cryptopp
kw:csp
kw:curl
kw:cutoff-date
kw:cycle
kw:cygwin
kw:d3
kw:daemon
kw:darcs
kw:darcsver
kw:database
kw:dataloss
kw:db
kw:dead-code
kw:deb
kw:debian
kw:debug
kw:deep-check
kw:defaults
kw:deferred
kw:delete
kw:deletion
kw:denial-of-service
kw:dependency
kw:deployment
kw:deprecation
kw:desert-island
kw:desert-island-build
kw:design
kw:design-review-needed
kw:detection
kw:dev-infrastructure
kw:devpay
kw:directory
kw:directory-page
kw:dirnode
kw:dirnodes
kw:disconnect
kw:discovery
kw:disk
kw:disk-backend
kw:distribute
kw:distutils
kw:dns
kw:do_http
kw:doc-needed
kw:docker
kw:docs
kw:docs-needed
kw:dokan
kw:dos
kw:download
kw:downloader
kw:dragonfly
kw:drop-upload
kw:duplicity
kw:dusty
kw:earth-dragon
kw:easy
kw:ec2
kw:ecdsa
kw:ed25519
kw:egg-needed
kw:eggs
kw:eliot
kw:email
kw:empty
kw:encoding
kw:endpoint
kw:enterprise
kw:enum34
kw:environment
kw:erasure
kw:erasure-coding
kw:error
kw:escaping
kw:etag
kw:etch
kw:evangelism
kw:eventual
kw:example
kw:excess-authority
kw:exec
kw:exocet
kw:expiration
kw:extensibility
kw:extension
kw:failure
kw:fedora
kw:ffp
kw:fhs
kw:figleaf
kw:file
kw:file-descriptor
kw:filename
kw:filesystem
kw:fileutil
kw:fips
kw:firewall
kw:first
kw:floatingpoint
kw:flog
kw:foolscap
kw:forward-compatibility
kw:forward-secrecy
kw:forwarding
kw:free
kw:freebsd
kw:frontend
kw:fsevents
kw:ftp
kw:ftpd
kw:full
kw:furl
kw:fuse
kw:garbage
kw:garbage-collection
kw:gateway
kw:gatherer
kw:gc
kw:gcc
kw:gentoo
kw:get
kw:git
kw:git-annex
kw:github
kw:glacier
kw:globalcaps
kw:glossary
kw:google-cloud-storage
kw:google-drive-backend
kw:gossip
kw:governance
kw:grid
kw:grid-manager
kw:gridid
kw:gridsync
kw:grsec
kw:gsoc
kw:gvfs
kw:hackfest
kw:hacktahoe
kw:hang
kw:hardlink
kw:heartbleed
kw:heisenbug
kw:help
kw:helper
kw:hint
kw:hooks
kw:how
kw:how-to
kw:howto
kw:hp
kw:hp-cloud
kw:html
kw:http
kw:https
kw:i18n
kw:i2p
kw:i2p-collab
kw:illustration
kw:image
kw:immutable
kw:impressions
kw:incentives
kw:incident
kw:init
kw:inlineCallbacks
kw:inotify
kw:install
kw:installer
kw:integration
kw:integration-test
kw:integrity
kw:interactive
kw:interface
kw:interfaces
kw:interoperability
kw:interstellar-exploration
kw:introducer
kw:introduction
kw:iphone
kw:ipkg
kw:iputil
kw:ipv6
kw:irc
kw:jail
kw:javascript
kw:joke
kw:jquery
kw:json
kw:jsui
kw:junk
kw:key-value-store
kw:kfreebsd
kw:known-issue
kw:konqueror
kw:kpreid
kw:kvm
kw:l10n
kw:lae
kw:large
kw:latency
kw:leak
kw:leasedb
kw:leases
kw:libgmp
kw:license
kw:licenss
kw:linecount
kw:link
kw:linux
kw:lit
kw:localhost
kw:location
kw:locking
kw:logging
kw:logo
kw:loopback
kw:lucid
kw:mac
kw:macintosh
kw:magic-folder
kw:manhole
kw:manifest
kw:manual-test-needed
kw:map
kw:mapupdate
kw:max_space
kw:mdmf
kw:memcheck
kw:memory
kw:memory-leak
kw:mesh
kw:metadata
kw:meter
kw:migration
kw:mime
kw:mingw
kw:minimal
kw:misc
kw:miscapture
kw:mlp
kw:mock
kw:more-info-needed
kw:mountain-lion
kw:move
kw:multi-users
kw:multiple
kw:multiuser-gateway
kw:munin
kw:music
kw:mutability
kw:mutable
kw:mystery
kw:names
kw:naming
kw:nas
kw:navigation
kw:needs-review
kw:needs-spawn
kw:netbsd
kw:network
kw:nevow
kw:new-user
kw:newcaps
kw:news
kw:news-done
kw:news-needed
kw:newsletter
kw:newurls
kw:nfc
kw:nginx
kw:nixos
kw:no-clobber
kw:node
kw:node-url
kw:notification
kw:notifyOnDisconnect
kw:nsa310
kw:nsa320
kw:nsa325
kw:numpy
kw:objects
kw:old
kw:openbsd
kw:openitp-packaging
kw:openssl
kw:openstack
kw:opensuse
kw:operation-helpers
kw:operational
kw:operations
kw:ophandle
kw:ophandles
kw:ops
kw:optimization
kw:optional
kw:options
kw:organization
kw:os
kw:os.abort
kw:ostrom
kw:osx
kw:osxfuse
kw:otf-magic-folder-objective1
kw:otf-magic-folder-objective2
kw:otf-magic-folder-objective3
kw:otf-magic-folder-objective4
kw:otf-magic-folder-objective5
kw:otf-magic-folder-objective6
kw:p2p
kw:packaging
kw:partial
kw:password
kw:path
kw:paths
kw:pause
kw:peer-selection
kw:performance
kw:permalink
kw:permissions
kw:persistence
kw:phone
kw:pickle
kw:pip
kw:pipermail
kw:pkg_resources
kw:placement
kw:planning
kw:policy
kw:port
kw:portability
kw:portal
kw:posthook
kw:pratchett
kw:preformance
kw:preservation
kw:privacy
kw:process
kw:profile
kw:profiling
kw:progress
kw:proxy
kw:publish
kw:pyOpenSSL
kw:pyasn1
kw:pycparser
kw:pycrypto
kw:pycrypto-lib
kw:pycryptopp
kw:pyfilesystem
kw:pyflakes
kw:pylint
kw:pypi
kw:pypy
kw:pysqlite
kw:python
kw:python3
kw:pythonpath
kw:pyutil
kw:pywin32
kw:quickstart
kw:quiet
kw:quotas
kw:quoting
kw:raic
kw:rainhill
kw:random
kw:random-access
kw:range
kw:raspberry-pi
kw:reactor
kw:readonly
kw:rebalancing
kw:recovery
kw:recursive
kw:redhat
kw:redirect
kw:redressing
kw:refactor
kw:referer
kw:referrer
kw:regression
kw:rekey
kw:relay
kw:release
kw:release-blocker
kw:reliability
kw:relnotes
kw:remote
kw:removable
kw:removable-disk
kw:rename
kw:renew
kw:repair
kw:replace
kw:report
kw:repository
kw:research
kw:reserved_space
kw:response-needed
kw:response-time
kw:restore
kw:retrieve
kw:retry
kw:review
kw:review-needed
kw:reviewed
kw:revocation
kw:roadmap
kw:rollback
kw:rpm
kw:rsa
kw:rss
kw:rst
kw:rsync
kw:rusty
kw:s3
kw:s3-backend
kw:s3-frontend
kw:s4
kw:same-origin
kw:sandbox
kw:scalability
kw:scaling
kw:scheduling
kw:schema
kw:scheme
kw:scp
kw:scripts
kw:sdist
kw:sdmf
kw:security
kw:self-contained
kw:server
kw:servermap
kw:servers-of-happiness
kw:service
kw:setup
kw:setup.py
kw:setup_requires
kw:setuptools
kw:setuptools_darcs
kw:sftp
kw:shared
kw:shareset
kw:shell
kw:signals
kw:simultaneous
kw:six
kw:size
kw:slackware
kw:slashes
kw:smb
kw:sneakernet
kw:snowleopard
kw:socket
kw:solaris
kw:space
kw:space-efficiency
kw:spam
kw:spec
kw:speed
kw:sqlite
kw:ssh
kw:ssh-keygen
kw:sshfs
kw:ssl
kw:stability
kw:standards
kw:start
kw:startup
kw:static
kw:static-analysis
kw:statistics
kw:stats
kw:stats_gatherer
kw:status
kw:stdeb
kw:storage
kw:streaming
kw:strports
kw:style
kw:stylesheet
kw:subprocess
kw:sumo
kw:survey
kw:svg
kw:symlink
kw:synchronous
kw:tac
kw:tahoe-*
kw:tahoe-add-alias
kw:tahoe-admin
kw:tahoe-archive
kw:tahoe-backup
kw:tahoe-check
kw:tahoe-cp
kw:tahoe-create-alias
kw:tahoe-create-introducer
kw:tahoe-debug
kw:tahoe-deep-check
kw:tahoe-deepcheck
kw:tahoe-lafs-trac-stream
kw:tahoe-list-aliases
kw:tahoe-ls
kw:tahoe-magic-folder
kw:tahoe-manifest
kw:tahoe-mkdir
kw:tahoe-mount
kw:tahoe-mv
kw:tahoe-put
kw:tahoe-restart
kw:tahoe-rm
kw:tahoe-run
kw:tahoe-start
kw:tahoe-stats
kw:tahoe-unlink
kw:tahoe-webopen
kw:tahoe.css
kw:tahoe_files
kw:tahoewapi
kw:tarball
kw:tarballs
kw:tempfile
kw:templates
kw:terminology
kw:test
kw:test-and-set
kw:test-from-egg
kw:test-needed
kw:testgrid
kw:testing
kw:tests
kw:throttling
kw:ticket999-s3-backend
kw:tiddly
kw:time
kw:timeout
kw:timing
kw:to
kw:to-be-closed-on-2011-08-01
kw:tor
kw:tor-protocol
kw:torsocks
kw:tox
kw:trac
kw:transparency
kw:travis
kw:travis-ci
kw:trial
kw:trickle
kw:trivial
kw:truckee
kw:tub
kw:tub.location
kw:twine
kw:twistd
kw:twistd.log
kw:twisted
kw:twisted-14
kw:twisted-trial
kw:twitter
kw:twn
kw:txaws
kw:type
kw:typeerror
kw:ubuntu
kw:ucwe
kw:ueb
kw:ui
kw:unclean
kw:uncoordinated-writes
kw:undeletable
kw:unfinished-business
kw:unhandled-error
kw:unhappy
kw:unicode
kw:unit
kw:unix
kw:unlink
kw:update
kw:upgrade
kw:upload
kw:upload-helper
kw:uri
kw:url
kw:usability
kw:use-case
kw:utf-8
kw:util
kw:uwsgi
kw:ux
kw:validation
kw:variables
kw:vdrive
kw:verify
kw:verlib
kw:version
kw:versioning
kw:versions
kw:video
kw:virtualbox
kw:virtualenv
kw:vista
kw:visualization
kw:visualizer
kw:vm
kw:volunteergrid2
kw:volunteers
kw:vpn
kw:wapi
kw:warners-opinion-needed
kw:warning
kw:weapi
kw:web
kw:web.port
kw:webapi
kw:webdav
kw:webdrive
kw:webport
kw:websec
kw:website
kw:websocket
kw:welcome
kw:welcome-page
kw:welcomepage
kw:wiki
kw:win32
kw:win64
kw:windows
kw:windows-related
kw:winscp
kw:workaround
kw:world-domination
kw:wrapper
kw:write-enabler
kw:wui
kw:x86
kw:x86-64
kw:xhtml
kw:xml
kw:xss
kw:zbase32
kw:zetuptoolz
kw:zfec
kw:zookos-opinion-needed
kw:zope
kw:zope.interface
p/blocker
p/critical
p/major
p/minor
p/normal
p/supercritical
p/trivial
r/cannot reproduce
r/duplicate
r/fixed
r/invalid
r/somebody else's problem
r/was already fixed
r/wontfix
r/worksforme
t/defect
t/enhancement
t/task
v/0.2.0
v/0.3.0
v/0.4.0
v/0.5.0
v/0.5.1
v/0.6.0
v/0.6.1
v/0.7.0
v/0.8.0
v/0.9.0
v/1.0.0
v/1.1.0
v/1.10.0
v/1.10.1
v/1.10.2
v/1.10a2
v/1.11.0
v/1.12.0
v/1.12.1
v/1.13.0
v/1.14.0
v/1.15.0
v/1.15.1
v/1.2.0
v/1.3.0
v/1.4.1
v/1.5.0
v/1.6.0
v/1.6.1
v/1.7.0
v/1.7.1
v/1.7β
v/1.8.0
v/1.8.1
v/1.8.2
v/1.8.3
v/1.8β
v/1.9.0
v/1.9.0-s3branch
v/1.9.0a1
v/1.9.0a2
v/1.9.0b1
v/1.9.1
v/1.9.2
v/1.9.2a1
v/cloud-branch
v/unknown
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: tahoe-lafs/trac#853
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Kai posted this bug report http://allmydata.org/pipermail/tahoe-dev/2009-December/003298.html
For what it is worth, we have a buildbot which tests Tahoe-LAFS on WinXP, and it works on that machine, so we have a starting point from which to figure out what is different between that build machine [1] and your machine.
Here is the report of the most recent five builds on the Windows buildbot:
http://allmydata.org/buildbot/builders/windows
And here is the dump of the versions of various components on that machine:
http://allmydata.org/buildbot/builders/windows/builds/1651/steps/show-tool-versions/logs/stdio
Kai: after you run the test and the test freezes, could you please look in the
_trial_temp
subdirectory and attach thetest.log
file to this ticket?Just out of curiousity, why are you using Tahoe-LAFS v1.5.0-r4108? The v1.5.0 release was -r4037 and the current head is -r4128. I guess you downloaded -r4108 back when it was the current head? I don't have any reason to think that this affects your bug.
So I've been looking at the code and I can't see how
res
can beNone
on [line 98 of iputil.py]source:src/allmydata/util/iputil.py@4108#L98._collect()
will be invoked and the argument passed to it will be whatever was returned from_find_addresses_via_config()
. Unless the thing that was returned fromfind_addresses_via_config()
is a Deferred, in which case the argument passed to_collect()
will be whatever value is given to that Deferred when it is fired. (This is the way Twisted works.)_find_addresses_via_config()
is defined on [source:src/allmydata/util/iputil.py@4108#L238 line 238]... Oh, okay I see where aNone
can be returned. From [source:src/allmydata/util/iputil.py@4108#L208 here]. Okay, so this suggests that on Kai's machine there is noroute.exe
executable on the path which emits a dotted-quad IPv4 address when executed asroute.exe print
.The first thing we need to do is make Tahoe-LAFS handle this case more gracefully. Let's see it is [source:src/allmydata/node.py@4108#L259] that invokes
iputil.get_local_address_async()
, and the result is passed to [source:src/allmydata/node.py@4108#L322 _setup_tub()]. What should we do when the answer to "list all of my IP addresses" is that there are none?In any case it seems like maybe [source:src/allmydata/util/iputil.py@4108#L194 iputil's SequentialTrier] should return an empty set instead of
None
when it runs out of executables to query.I'm investigating a cryptopp problem on my Windows Seven 64, using official Mingw-5.1.6, which shows up as a WebResultsRendering never ending test_check.
I had to manually install pycryptopp patching his config.h, disabling assembler parts (#define CRYPTOPP_DISABLE_ASM) in order to pass tahoe tests.
Kai, could you please run this command and report if it loops to infinite :
Are you using a 64-bits XP ? What is your processor ?
Thanks.
zooko: We are using r4108 because it is the latest generated tarball (builder tarballs seems defunct since 21^th^ November).
Please help me diagnose this pycryptopp problem. What version of pycryptopp?
There have been some bugs in amd64 Crypto++ assembly recently but all known problems are fixed in the most recent release of pycryptopp. There could be an interaction with Mingw -- I don't think many people have used Mingw to build pycryptopp before.
The tarballs stopped coming because some of the Supported Platforms stopped passing unit tests after r4108. That means that r4108 is the most recent version for which all the Supported Platforms passed all unit tests. r4108 is fine for now.
Attachment test.log (496 bytes) added
_trial_tmp\test.log
Replying to zooko:
I have no experience to use darcs. So I DLed the latest package in http://allmydata.org/source/tahoe/tarballs/ , that is, allmydata-tahoe-1.5.0-r4108.tar.bz2 .
Replying to zooko:
...
Oops. I set
PATH=C:\mingw516\bin
, which resulted in excluding route.exe from PATH!As I had already installed cygwin tools including g++ and python, I thought that it was better to exclude them from PATH not to confuse build process, and that setting
PATH=C:\mingw516\bin
was the easiest way to do it.After installation of Tahoe with
PATH=C:\mingw516\bin
, I started a new shell without changing PATH and started Tahoe with it. Tahoe successfully started and I could access the WUI with firefox. However, when I tried to DL a JPG file found in http://testgrid.allmydata.org:3567/ via my WUI, it freezes again and no access to my WUI was possible, including the access to the main screen. At this time, python.exe occupies ~100 % CPU.Replying to Grumpf:
At this time, python occupied ~100 % CPU and had to terminated by Task Manager. The content of _trial_tmp\test.log was:
The above test was done on my PC in my home. WinXP Pro SP3 32 bit is installed, as the CPU is PenM 1.2 GHz. When I first reported the problem, it was on the PC in my office. So I don't remember the detailed specs about the OS (sorry,) but probably it was WinXP Pro SP3 32 bit running on Core2Duo E7400.
Replying to zooko:
According to the WUI just after startup,
My versions
allmydata-tahoe: 1.5.0-r4108, foolscap: 0.4.2, pycryptopp: 0.5.17, zfec: 1.4.5, Twisted: 8.2.0, Nevow: 0.9.33, zope.interface: 3.5.2, python: 2.6.4, platform: Windows-XP-5.1.2600-SP3, sqlite: 3.5.9, simplejson: 2.0.9, pyopenssl: 0.9, argparse: 0.9.1, twisted: 8.2.0, nevow: 0.9.33-r17222, pyOpenSSL: 0.9, pyutil: 1.3.34, zbase32: 1.1.1, setuptools: 0.6c12dev, pysqlite: 2.4.1
Okay I moved the issue about
route.exe
not being found over to a new ticket: #854 (what to do when you can't find any IP address for yourself).This ticket is now renamed to "unit tests hang on Windows (pycryptopp related?)". Let me see if I understand what has been reported so far:
allmydata.test.test_checker.WebResultsRendering.test_check
using the official "python.org" Python 2.6.4 (from http://www.python.org/download/ ) on Windows XP SP3 32-bit. Histest.log
doesn't have anything useful in it. Kai: would you please turn on verbose logging ( http://allmydata.org/trac/tahoe/browser/docs/logging.txt ) and try again?config.h
? Does Tahoe-LAFS fail to build? Does it build but fail its unit tests? If so in what way? Also could you please run the pycryptopp unit tests.networking failures on WinXPto unit tests hang on Windows (pycryptopp related?)Replying to zooko:
I couldn't turn on verbose logging mode either by the arguments of make, nor by
SET FLOGTOTWISTED=1
etc. The content of test.log was same as before.I also tried
SET FLOGFILE=flog.out
, only to find empty flog.out.Zooko: Could you tell me what am I doing wrong with this, or how to hard-code the verbose mode flag into some .py files ?
(Instead I manually inserted
print inspect.stack()[0][1:3]
after each line ofWebResultsRendering.test_check
and saw thatu = uri.CHKFileURI("\x00"*16, "\x00"*32, 3, 10, 1234)
causes hangup. And with respect toHashUtilTests
, the cause wash1 = hashutil.convergence_hash(3, 10, 1000, "data", "secret")
. But I didn't investigate further ...)Additionally, when I DLed and built pycryptopp independently from Tahoe, the unit test aborted:
At this time, the "sorry for the inconvenience" dialog appeared.
Moreover, the unit test of Crypto++ also aborted during the test of SHA-256.
It might be that Crypto++ doesn't compile with Mingw, but I'm not sure which part of the programs is responsible for our problem.
Working crappy workaround is to downgrade GNU as from 2.20 (current MinGW) to previous 2.19.1 (both as.exe files and nothing else at all).
The problem is inherited from Crypto++.
Now to figure if it is a MinGW-centric problem, and a real fix...
Zooko: I suffer the exact same problems nodakai do. I pinned it down to Crypto++, symptoms are never-ending Tahoe test units, crash during first pycrypto test unit and never-ending Crypto++ SHA256 test.
Same underlying issue as (@@http://www.mail-archive.com/cryptopp-users@googlegroups.com/msg05292.html@@) ?
Maybe the fix mentioned in (@@http://www.mail-archive.com/cryptopp-users@googlegroups.com/msg05299.html@@) didn't make it into as 2.20.
Replying to davidsarah:
Sorry, the regression was in 2.20. It is fixed in some branch of 2.20, but not released.
Removing windows tag since this is known to happen also on Ubuntu Linux (https://bugs.launchpad.net/ubuntu/+bug/461303) when using the buggy as.
The pycryptopp bug is http://allmydata.org/trac/pycryptopp/ticket/31 .unit tests hang on Windows (pycryptopp related?)to pycryptopp-related hang of unit tests on platforms using buggy Gnu as 2.20 (e.g. MinGW 5.1.x, Ubuntu Karmic)Here is the launchpad page which tracks this issue across all of the various operating systems and projects that it affects:
https://bugs.launchpad.net/ubuntu/+source/binutils/+bug/461303
The as bug is x86-specific. It's not clear whether it affects only x86-64, or also x86-32.
Hrm. So we could work-around this by releasing a new version of pycryptopp which inspects the version number of GNU assembler and the platform and tries to figure out if this particular version of GNU assembler has this bug, and if so then disable the assembly optimizations. Or, it could even be more clever and try compiling and executing, and if it segfaults then it turns off assembly optimization and tries again. But I think instead we should just close this as 'invalid', indicating that it is the problem of some other people (a combination of binutils to produce a new fixed release -- http://sourceware.org/bugzilla/show_bug.cgi?id=10856 -- and/or MingW to patch binutils or to upgrade binutils in MingW to the fixed release -- https://sourceforge.net/tracker/index.php?func=detail&aid=2913876&group_id=2435&atid=102435 ).
Replying to zooko:
I don't entirely agree. This isn't the first time that pycryptopp/Crypto++ has failed in a platform-dependent way that was detected by unit tests -- see pycryptopp#17 and pycryptopp#24. If pycryptopp (or Crypto++) were to do the "more clever" behaviour above, then we'd have a good chance of heading off such problems in future. (Some bugs would be prevented, others would fail compilation rather than failing only if unit tests are run explicitly.) But that can be a separate ticket.
Well, the idea I floated above of attempting compilation, then test, then recompilation a different way if the test fails makes me think it is "too clever". I could imagine a bug in that process leading to problems even when there isn't a bug in Crypto++! On the other hand, I definitely think we should add a "quick start up self test" to pycryptopp and in fact I have already done so but I haven't committed to to pycryptopp trunk yet. Once that is in place then Tahoe-LAFS can import pycryptopp, execute the quick start-up self-test, and if it fails Tahoe-LAFS can exit quickly and loudly.
What do you think?
By the way there is an idiom in Python packaging which is to try compiling a native-code extension module and if the compilation fails (such as if there is no C compiler present, or some necessary header files are missing), then go ahead and use the pure-Python implementation of that module instead. So maybe if we wanted to go down this route we could start by adapting that idiom.
Replying to zooko:
+1.
FreeStorm also hit this bug, on Windows XP with MinGW. The symptoms were slightly different: when running the pycryptopp tests using
python setup.py test
, the error wasThis was solved by downgrading to binutils 2.19.1 -- specifically, by extracting binutils-2.19.1-mingw32-bin.tar.gz over the MinGW installation directory.
Replying to davidsarah:
AdvancedInstall#Whatifthatdoesntwork has been updated to describe this.
I just tested this on MinGW, and it looks like the issue has been fixed there (probably because they upgraded to the latest snapshot of binutils, version 2.20.51.20100613, to fix an unrelated bug).
Thanks, Wei Dai. I've updated wiki/AdvancedInstall#Whatifthatdoesntwork.
binutils 2.20.1, released 2010-03-03, does not have the ChangeLog entry from http://sourceware.org/bugzilla/show_bug.cgi?id=10856#c5 but does have the patch to expr.c. Weird. But I guess it is fixed in binutils 2.20.1.