Chromium Code Reviews
Help | Chromium Project | Sign in
(54)

Issue 5552005: OpenSSL <=> PKCS#11 binding. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 5 months ago by Nelson Araujo
Modified:
4 years ago
Reviewers:
James Simonsen
CC:
chromium-os-reviews_chromium.org, msb+crosoverlay_chromium.org, adlr, anush
Visibility:
Public.

Description

OpenSSL <=> PKCS#11 binding. Change-Id: I16c17528e1dd3711c08087ed330be60a73a32ebd BUG=chromium-os:331 TEST=test entd:crypto-openssl-newcsr.js passed. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=c22a2f4

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -0 lines) Patch
A dev-libs/openssl/files/openssl-pkcs11-engine.patch View 1 chunk +24 lines, -0 lines 0 comments Download
M dev-libs/openssl/openssl-0.9.8o.ebuild View 1 chunk +1 line, -0 lines 0 comments Download
Trybot results:
Commit: CQ not working?

Messages

Total messages: 4 (0 generated)
Nelson Araujo
4 years, 5 months ago (2010-12-07 18:18:44 UTC) #1
James Simonsen
I'm okay with the config. I think we should double check with someone who's more ...
4 years, 5 months ago (2010-12-07 22:34:22 UTC) #2
James Simonsen
LGTM
4 years, 5 months ago (2010-12-08 01:02:34 UTC) #3
Nelson Araujo
4 years, 5 months ago (2010-12-08 02:20:19 UTC) #4
On 2010/12/07 22:34:22, James Simonsen wrote:
> I'm okay with the config. I think we should double check with someone who's
more
> experienced with CrOS to see how we're supposed to distribute modified config
> files. Could you find  any examples from other packages?

Did check with OS devs and the chromiumos-overlay would be the right place to
make the change.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be