On 19-10-2010 19:23, Jeffrey Walton wrote:
So I wasted my precious time preparing a patch while someone
else had already posted a patch off-list.
Lol... If you're going to throw a tantrum every time someone beats you
ta a patch, you owe us a tantrum:

WinCE patch:
http://www.mail-archive.com/openssl-users@openssl.org/msg61765.html
Pierre Delaage submission:
http://www.mail-archive.com/openssl-...@openssl.org/msg28264.html

:)

I don't mind someone patching it before me, that is fine.

I mind that someone moved a discussion off-list without cross-posting to the original list or at least posting a FUT-like message to the original thread, specifying where exactly the discussion has been moved. This is general good practice, also to help those who might (much later) find the thread in Google and get only
half the story.

(The post suggesting that someone file a bug is not an exact FUT, a link to the bug number as soon
as work began would have been).

On Tue, Oct 19, 2010 at 7:20 AM, Jakob Bohm<jb-open...@wisemo.com>  wrote:
  On 19-10-2010 12:32, per frykenvall wrote:
  Thanks, Jakob! However, I followed Dr. Stephen Hensons advice earlier in
this thread and filed a report to the bug tracker, and got a resolution from
Andy Polyakov a week ago, based on your suggestion. I've tested it and am
fully satisfied:

[openssl.org #2356] Resolved: LNK4078 and LNK4210 linking with x64 static
libs
http://cvs.openssl.org/chngview?cn=19935

So I wasted my precious time preparing a patch while someone else had
already posted a
patch off-list.

Thanks to everyone involved for not telling the list that this issue had
been resolved in another forum!


Of course, that resolution does not include the race condition you
describe.

Best regards,
Per

On 2010-10-18 17:35, Jakob Bohm wrote:
I have now created an actual patch to fix this.
It turns out to be a small pattern bug in x86_64xlate.pl

Patch attached as openssl-1.0.0a-x86_64attr.patch.

While debugging this patch I ran into an unrelated issue where nmake
would invoke nasm before the .asm file had been completely output.
This is probably a bug in the perl build used on one of the test
machines, but I think the patch to kludge around that race condition
might be useful too.

Patch attached as openssl-1.0.0a-x86_64cpuid-build-race.patch.

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
User Support Mailing List                    openssl-users@openssl.org
Automated List Manager                           majord...@openssl.org

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
User Support Mailing List                    openssl-users@openssl.org
Automated List Manager                           majord...@openssl.org

Reply via email to