Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#22680 closed defect (fixed)

bsdsfv doesn't work correctly when compiled 64-bit

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: simmel@…
Priority: Normal Milestone:
Component: ports Version: 1.8.1
Keywords: snowleopard Cc: 0xced (Cédric Luthi)
Port: bsdsfv

Description

bsdsfv doesn't work properly when compiled 64-bit (as is the default on Snow Leopard). I filed an upstream bug report but they have not reacted to it, nor the other bug reports that I see have been reported over the years, so I guess the project is dead. Perhaps a local fix can be developed for MacPorts.

Or, perhaps the cksfv port works better. It has been updated more recently. If so, perhaps we should mark the bsdsfv port "replaced_by cksfv".

Attachments (1)

bsdsfv.diff (670 bytes) - added by ryandesign (Ryan Carsten Schmidt) 14 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 14 years ago by simmel@…

Hm, can you make a port only compile itself in "32-bit mode"?

I hate to say "works for me", but I have no 64-bit only system yet so bsdsfv still works for me.

Saying that cksfv replaces bsdsfv is harsh. There are several tools in MacPorts which solve the same problem, csfv and bsdsfv are just one example.

comment:2 Changed 14 years ago by simmel@…

Maybe build-arch can be used in the Portsfile. I'll try it later.

comment:3 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

No problem keeping bsdsfv, if it can either be made to work properly in 64-bit mode or forced to 32-bit mode. The attached patch does the latter, and makes sure it's always UsingTheRightCompiler. Does this look ok to you?

Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: bsdsfv.diff added

comment:4 Changed 14 years ago by simmel@…

Looks awesome!

I can't change this ticket into an update ticket, so if you can change the type do it or you'll have to create a new one.

Thanks for reporting and fixing this, being a maintainer have never been easier ; )

Have a nice weekend!

comment:5 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Committed in r60932! Hope you have a good weekend too.

comment:6 Changed 14 years ago by tobypeterson

Really simple issue, committed r62904

Note: See TracTickets for help on using tickets.