Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] PATCH > To integrate pycrypto into Buildroot
@ 2012-11-28  5:05 Noel Roberts
  2012-11-28 13:02 ` Thomas Petazzoni
  2012-11-28 21:12 ` Arnout Vandecappelle
  0 siblings, 2 replies; 5+ messages in thread
From: Noel Roberts @ 2012-11-28  5:05 UTC (permalink / raw)
  To: buildroot


This patch allows Buildroot to cross compile the crypto libraries needed for python integration, enabling good cryptographic support within Python.

This is my first patch submission - please advise me, if I need to rework anything.

[Below is the original developer?s permission for inclusion]

Regards
Noel

From: Dwayne Litzenberger [mailto:dlitz at dlitz.net]
Sent: Wednesday, 28 November 2012 9:01 AM
To: Noel Roberts
Subject: Re: Including pycrypto in Buildroot

You don't need my permission. Go nuts!

Out of curiosity, what uses PyCryptoin buildroot?

Noel Roberts <Noel.Roberts at brisbane.qld.gov.au<mailto:Noel.Roberts@brisbane.qld.gov.au>> wrote:
Good day

Would you object to me submitting a patch to buildroot, to allow the inclusion of pycrypto within the buildroot environment?

Regards
Noel


**************************************************** This message has passed through an insecure network. Please direct all enquiries to the message author. ****************************************************

**************************************************** This message has passed through an insecure network. Please direct all enquiries to the message author. ****************************************************

****************************************************
This message has passed through an insecure network.
Please direct all enquiries to the message author.
****************************************************
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121128/5d0f509c/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Adding-pyCrypto-functionality-in-Buildroot.patch
Type: application/octet-stream
Size: 1759 bytes
Desc: 0001-Adding-pyCrypto-functionality-in-Buildroot.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121128/5d0f509c/attachment.obj>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] PATCH > To integrate pycrypto into Buildroot
  2012-11-28  5:05 [Buildroot] PATCH > To integrate pycrypto into Buildroot Noel Roberts
@ 2012-11-28 13:02 ` Thomas Petazzoni
  2012-11-28 21:12 ` Arnout Vandecappelle
  1 sibling, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2012-11-28 13:02 UTC (permalink / raw)
  To: buildroot

Dear Noel Roberts,

On Wed, 28 Nov 2012 05:05:29 +0000, Noel Roberts wrote:

> This patch allows Buildroot to cross compile the crypto libraries
> needed for python integration, enabling good cryptographic support
> within Python.

Good. The part adding the package in package/Config.in is missing.

> This is my first patch submission - please advise me, if I need to
> rework anything.

Could you use git send-email to send your patch?

Thanks!

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] PATCH > To integrate pycrypto into Buildroot
  2012-11-28  5:05 [Buildroot] PATCH > To integrate pycrypto into Buildroot Noel Roberts
  2012-11-28 13:02 ` Thomas Petazzoni
@ 2012-11-28 21:12 ` Arnout Vandecappelle
  2012-11-30  2:27   ` Noel Roberts
  1 sibling, 1 reply; 5+ messages in thread
From: Arnout Vandecappelle @ 2012-11-28 21:12 UTC (permalink / raw)
  To: buildroot

On 28/11/12 06:05, Noel Roberts wrote:
> This patch allows Buildroot to cross compile the crypto libraries needed for python integration, enabling good
> cryptographic support within Python.
>
> This is my first patch submission - please advise me, if I need to rework anything.

  We prefer patches to be sent in-line (with git send-email) because then it is
easier to comment on them.  But I'll already give some feedback before you re-post.

- If you include it in package/Config.in's 'external python modules' menu,
then the 'depends on BR2_PACKAGE_PYTHON' is redundant.

- Please add the license information:
PYTHON_CRYPTO_LICENSE = Public domain, Python 2.2 license
PYTHON_CRYPTO_LICENSE_FILES = COPYRIGHT LEGAL/copy/LICENSE.python-2.2

- Since there is a check for wchar.h in configure.ac, I would add an
explicit 'depends on BR2_USE_WCHAR', even though python itself already
depends on it. But who knows, maybe that will ever be removed.


  Regards,
  Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] PATCH > To integrate pycrypto into Buildroot
  2012-11-28 21:12 ` Arnout Vandecappelle
@ 2012-11-30  2:27   ` Noel Roberts
  2012-11-30  5:21     ` Samuel Martin
  0 siblings, 1 reply; 5+ messages in thread
From: Noel Roberts @ 2012-11-30  2:27 UTC (permalink / raw)
  To: buildroot

Good day Arnout

Grrrr, git send-email has issues with my virtual machine/proxy configuration. Not getting decent error messages and googling didn't help either. Not sure which part isn't working, just that SMPT couldn't initialize correctly.

Attached are the newest patch files.



-----Original Message-----
From: Arnout Vandecappelle [mailto:arnout at mind.be] 
Sent: Thursday, 29 November 2012 7:13 AM
To: Noel Roberts
Cc: buildroot at uclibc.org
Subject: Re: [Buildroot] PATCH > To integrate pycrypto into Buildroot

On 28/11/12 06:05, Noel Roberts wrote:
> This patch allows Buildroot to cross compile the crypto libraries 
> needed for python integration, enabling good cryptographic support within Python.
>
> This is my first patch submission - please advise me, if I need to rework anything.

  We prefer patches to be sent in-line (with git send-email) because then it is easier to comment on them.  But I'll already give some feedback before you re-post.

- If you include it in package/Config.in's 'external python modules' menu, then the 'depends on BR2_PACKAGE_PYTHON' is redundant.

- Please add the license information:
PYTHON_CRYPTO_LICENSE = Public domain, Python 2.2 license PYTHON_CRYPTO_LICENSE_FILES = COPYRIGHT LEGAL/copy/LICENSE.python-2.2

- Since there is a check for wchar.h in configure.ac, I would add an explicit 'depends on BR2_USE_WCHAR', even though python itself already depends on it. But who knows, maybe that will ever be removed.


  Regards,
  Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

****************************************************
This message has passed through an insecure network.
Please direct all enquiries to the message author.
****************************************************

****************************************************
This message has passed through an insecure network.
Please direct all enquiries to the message author.
****************************************************
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Adding-pyCrypto-functionality-in-Buildroot.patch
Type: application/octet-stream
Size: 1883 bytes
Desc: 0001-Adding-pyCrypto-functionality-in-Buildroot.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121130/5330b901/attachment-0003.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Added-commentator-feedback.patch
Type: application/octet-stream
Size: 1465 bytes
Desc: 0002-Added-commentator-feedback.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121130/5330b901/attachment-0004.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Added-missing-Config.in-in-git-commit.patch
Type: application/octet-stream
Size: 810 bytes
Desc: 0003-Added-missing-Config.in-in-git-commit.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121130/5330b901/attachment-0005.obj>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Buildroot] PATCH > To integrate pycrypto into Buildroot
  2012-11-30  2:27   ` Noel Roberts
@ 2012-11-30  5:21     ` Samuel Martin
  0 siblings, 0 replies; 5+ messages in thread
From: Samuel Martin @ 2012-11-30  5:21 UTC (permalink / raw)
  To: buildroot

Hi Noel,

2012/11/30 Noel Roberts <Noel.Roberts@brisbane.qld.gov.au>:
> Good day Arnout
>
> Grrrr, git send-email has issues with my virtual machine/proxy configuration. Not getting decent error messages and googling didn't help either. Not sure which part isn't working, just that SMPT couldn't initialize correctly.
>
To use git send-email you may need some additional Perl packages:
net-smtp-ssl, authen-sasl.
(see the Example section in:
http://www.kernel.org/pub/software/scm/git/docs/git-send-email.html)


Regards,

-- 
Sam

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-11-30  5:21 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-28  5:05 [Buildroot] PATCH > To integrate pycrypto into Buildroot Noel Roberts
2012-11-28 13:02 ` Thomas Petazzoni
2012-11-28 21:12 ` Arnout Vandecappelle
2012-11-30  2:27   ` Noel Roberts
2012-11-30  5:21     ` Samuel Martin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox