search

sdague / amt

Python tools for interacting with Intel's AMT hardware control interfaces
Apache License 2.0
65 stars 30 forks source link

python 3 conversion #8

Closed Rixillo closed 8 years ago

Rixillo commented 8 years ago

I finally accomplished building amt0.30 on CentOS7 and Fedora24. I had similar difficulties on both platforms, mainly due to the lack of available python3 prerequisites. Also, it seems that the amt package has been converted to python3 but there are still some remnants of py2 that caused it fail either at build time or run time.

I made several attempts to either build the prerequisites or pull them from various repositories, so I am a little confused now about which of various sources actually went into the final successful build of amt. However, I used the CentOS "ius" repository, "IUS Community Packages for Enterprise Linux 7 - x86_64" to install python35u and I used the Openwsman repository, http://download.opensuse.org/repositories/Openwsman/CentOS_CentOS-7/Openwsman.repo, to install libwsman3.

I needed to fix a couple of python2isms, which caused build or run-time fatalities on both CentOS7 and Fedora24. Here are the diffs:

client.py.patch.txt hostdb.py.patch.txt

Thanks for this package. I've been crying ever since they broke my amttool. Rix

sdague commented 8 years ago

On 10/04/2016 08:37 PM, Rixillo wrote:

I finally accomplished building amt0.30 on CentOS7 and Fedora24. I had similar difficulties on both platforms, mainly due to the lack of available python3 prerequisites. Also, it seems that the amt package has been converted to python3 but there are still some remnants of py2 that caused it fail either at build time or run time.

I made several attempts to either build the prerequisites or pull them from various repositories, so I am a little confused now about which of various sources actually went into the final successful build of amt. However, I used the CentOS "ius" repository, "IUS Community Packages for Enterprise Linux 7 - x86_64" to install python35u and I used the Openwsman repository, http://download.opensuse.org/repositories/Openwsman/CentOS_CentOS-7/Openwsman.repo, to install libwsman3.

I needed to fix a couple of python2isms, which caused build or run-time fatalities on both CentOS7 and Fedora24. Here are the diffs:

client.py.patch.txt https://github.com/sdague/amt/files/509786/client.py.patch.txt hostdb.py.patch.txt https://github.com/sdague/amt/files/509787/hostdb.py.patch.txt

Thanks for this package. I've been crying ever since they broke my amttool. Rix

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA4H3LiYNslf01_Ab8haxEAtWdul443ks5qwvFXgaJpZM4KOTv4.

Is it possible to do this as a pull request instead of attached patches?

-Sean

Sean Dague http://dague.net

Rixillo commented 8 years ago

Sean,

I am an old-timer perl programmer and I still am learning new stuff like python and git. I will try to figure out how to do that. Thanks.

Richard "Rix" Ketcham

On 2016-10-05 03:33, Sean Dague wrote:

On 10/04/2016 08:37 PM, Rixillo wrote:

I finally accomplished building amt0.30 on CentOS7 and Fedora24. I had similar difficulties on both platforms, mainly due to the lack of available python3 prerequisites. Also, it seems that the amt package has been converted to python3 but there are still some remnants of py2 that caused it fail either at build time or run time.

I made several attempts to either build the prerequisites or pull them from various repositories, so I am a little confused now about which of various sources actually went into the final successful build of amt. However, I used the CentOS "ius" repository, "IUS Community Packages for Enterprise Linux 7 - x86_64" to install python35u and I used the Openwsman repository,

http://download.opensuse.org/repositories/Openwsman/CentOS_CentOS-7/Openwsman.repo, to install libwsman3.

I needed to fix a couple of python2isms, which caused build or run-time fatalities on both CentOS7 and Fedora24. Here are the diffs:

client.py.patch.txt https://github.com/sdague/amt/files/509786/client.py.patch.txt hostdb.py.patch.txt https://github.com/sdague/amt/files/509787/hostdb.py.patch.txt

Thanks for this package. I've been crying ever since they broke my amttool. Rix

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8, or mute the thread

https://github.com/notifications/unsubscribe-auth/AAA4H3LiYNslf01_Ab8haxEAtWdul443ks5qwvFXgaJpZM4KOTv4.

Is it possible to do this as a pull request instead of attached patches?

-Sean

Sean Dague http://dague.net

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8#issuecomment-251639362, or mute the thread https://github.com/notifications/unsubscribe-auth/AVlDyA2tjAjJPC1P4i5ksWAXnNoRfyxGks5qw3zegaJpZM4KOTv4.

sdague commented 8 years ago

Ok, no problem. If you can get it sorted this week, I can merge it as a PR. Otherwise I'll try to carve out a bit of time next week to get this sorted and get the python 3 testing running upstream.

-Sean

On 10/05/2016 10:22 AM, Rixillo wrote:

Sean,

I am an old-timer perl programmer and I still am learning new stuff like python and git. I will try to figure out how to do that. Thanks.

Richard "Rix" Ketcham

On 2016-10-05 03:33, Sean Dague wrote:

On 10/04/2016 08:37 PM, Rixillo wrote:

I finally accomplished building amt0.30 on CentOS7 and Fedora24. I had similar difficulties on both platforms, mainly due to the lack of available python3 prerequisites. Also, it seems that the amt package has been converted to python3 but there are still some remnants of py2 that caused it fail either at build time or run time.

I made several attempts to either build the prerequisites or pull them from various repositories, so I am a little confused now about which of various sources actually went into the final successful build of amt. However, I used the CentOS "ius" repository, "IUS Community Packages for Enterprise Linux 7 - x86_64" to install python35u and I used the Openwsman repository,

http://download.opensuse.org/repositories/Openwsman/CentOS_CentOS-7/Openwsman.repo,

to install libwsman3.

I needed to fix a couple of python2isms, which caused build or run-time fatalities on both CentOS7 and Fedora24. Here are the diffs:

client.py.patch.txt https://github.com/sdague/amt/files/509786/client.py.patch.txt hostdb.py.patch.txt https://github.com/sdague/amt/files/509787/hostdb.py.patch.txt

Thanks for this package. I've been crying ever since they broke my amttool. Rix

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8, or mute the thread

https://github.com/notifications/unsubscribe-auth/AAA4H3LiYNslf01_Ab8haxEAtWdul443ks5qwvFXgaJpZM4KOTv4.

Is it possible to do this as a pull request instead of attached patches?

-Sean

Sean Dague http://dague.net

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8#issuecomment-251639362, or mute the thread

https://github.com/notifications/unsubscribe-auth/AVlDyA2tjAjJPC1P4i5ksWAXnNoRfyxGks5qw3zegaJpZM4KOTv4.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8#issuecomment-251689175, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA4H2EVDD4HnBGEMkriQixasWB-fv6fks5qw7KYgaJpZM4KOTv4.

Sean Dague http://dague.net

Rixillo commented 8 years ago

Thanks

Rix

On 2016-10-05 07:24, Sean Dague wrote:

Ok, no problem. If you can get it sorted this week, I can merge it as a PR. Otherwise I'll try to carve out a bit of time next week to get this sorted and get the python 3 testing running upstream.

-Sean

On 10/05/2016 10:22 AM, Rixillo wrote:

Sean,

I am an old-timer perl programmer and I still am learning new stuff like python and git. I will try to figure out how to do that. Thanks.

Richard "Rix" Ketcham

On 2016-10-05 03:33, Sean Dague wrote:

On 10/04/2016 08:37 PM, Rixillo wrote:

I finally accomplished building amt0.30 on CentOS7 and Fedora24. I had similar difficulties on both platforms, mainly due to the lack of available python3 prerequisites. Also, it seems that the amt package has been converted to python3 but there are still some remnants of py2 that caused it fail either at build time or run time.

I made several attempts to either build the prerequisites or pull them from various repositories, so I am a little confused now about which of various sources actually went into the final successful build of amt. However, I used the CentOS "ius" repository, "IUS Community Packages for Enterprise Linux 7 - x86_64" to install python35u and I used the Openwsman repository,

http://download.opensuse.org/repositories/Openwsman/CentOS_CentOS-7/Openwsman.repo,

to install libwsman3.

I needed to fix a couple of python2isms, which caused build or run-time fatalities on both CentOS7 and Fedora24. Here are the diffs:

client.py.patch.txt https://github.com/sdague/amt/files/509786/client.py.patch.txt hostdb.py.patch.txt https://github.com/sdague/amt/files/509787/hostdb.py.patch.txt

Thanks for this package. I've been crying ever since they broke my amttool. Rix

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8, or mute the thread

https://github.com/notifications/unsubscribe-auth/AAA4H3LiYNslf01_Ab8haxEAtWdul443ks5qwvFXgaJpZM4KOTv4.

Is it possible to do this as a pull request instead of attached patches?

-Sean

Sean Dague http://dague.net

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8#issuecomment-251639362, or mute the thread

https://github.com/notifications/unsubscribe-auth/AVlDyA2tjAjJPC1P4i5ksWAXnNoRfyxGks5qw3zegaJpZM4KOTv4.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8#issuecomment-251689175, or mute the thread

https://github.com/notifications/unsubscribe-auth/AAA4H2EVDD4HnBGEMkriQixasWB-fv6fks5qw7KYgaJpZM4KOTv4.

Sean Dague http://dague.net

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/sdague/amt/issues/8#issuecomment-251689692, or mute the thread https://github.com/notifications/unsubscribe-auth/AVlDyM4gBwcFlyt-VO41KH5Egr0Wcp57ks5qw7MBgaJpZM4KOTv4.