[ITP] python-getdevinfo

classic Classic list List threaded Threaded
27 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[ITP] python-getdevinfo

cygwin-apps mailing list
Hello,

This email signals my intent to package getdevinfo.

(https://www.hamishmb.com/html/downloads.php?program_name=getdevinfo)
for Cygwin.

getdevinfo is module I wrote for collecting device information, and is a
dependency for DDRescue-GUI, which I intend to package next if this is
approved.

Once I have done these things, I intend to improve Cygwin's device
information detection capabilities and release a further update to make
this more complete.

If anyone has feedback I'd appreciate it very much.

Hamish McIntyre-Bhatty




0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 17/08/2020 14:33, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> Hello,
>
> This email signals my intent to package getdevinfo.
>
> (https://www.hamishmb.com/html/downloads.php?program_name=getdevinfo)
> for Cygwin.
>
> getdevinfo is module I wrote for collecting device information, and is a
> dependency for DDRescue-GUI, which I intend to package next if this is
> approved.
>
> Once I have done these things, I intend to improve Cygwin's device
> information detection capabilities and release a further update to make
> this more complete.
>
> If anyone has feedback I'd appreciate it very much.
>
> Hamish McIntyre-Bhatty
>
Forgot link to packages: https://www.hamishmb.com/files/cygwin-temp/

0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list


On 17.08.2020 15:34, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> On 17/08/2020 14:33, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> Hello,
>>
>> This email signals my intent to package getdevinfo.
>>
>> (https://www.hamishmb.com/html/downloads.php?program_name=getdevinfo)
>> for Cygwin.
>>
>> getdevinfo is module I wrote for collecting device information, and is a
>> dependency for DDRescue-GUI, which I intend to package next if this is
>> approved.
>>
>> Once I have done these things, I intend to improve Cygwin's device
>> information detection capabilities and release a further update to make
>> this more complete.
>>
>> If anyone has feedback I'd appreciate it very much.
>>
>> Hamish McIntyre-Bhatty
>>
> Forgot link to packages: https://www.hamishmb.com/files/cygwin-temp/
>

something looks wrong on test

======================================================================
ERROR: test_get_info (tests.getdevinfo_tests_cygwin.TestGetInfo)
Test that the information can be collected on this system without error
----------------------------------------------------------------------
Traceback (most recent call last):
   File
"/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/src/getdevinfo-1.1.0/getdevinfo/tests/getdevinfo_tests_cygwin.py",
line 218, in test_get_info
     cygwin.get_info()
   File
"/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
line 101, in get_info
     get_device_info(disk)
   File
"/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
line 135, in get_device_info
     cmd = subprocess.run([SMARTCTL, "-i", host_disk, "-j"],
stdout=subprocess.PIPE,
   File "/usr/lib/python3.8/subprocess.py", line 489, in run
     with Popen(*popenargs, **kwargs) as process:
   File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
     self._execute_child(args, executable, preexec_fn, close_fds,
   File "/usr/lib/python3.8/subprocess.py", line 1637, in _execute_child
     self.pid = _posixsubprocess.fork_exec(
BlockingIOError: [Errno 11] Resource temporarily unavailable

----------------------------------------------------------------------
Ran 23 tests in 0.679s

FAILED (errors=1)
NOTE: These tests won't work correctly without administrator privileges.

$ id
uid=197609(Marco) gid=544(Administratoren)
groups=544(Administratoren),197121(Kein)
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 19/08/2020 06:53, Marco Atzeri via Cygwin-apps wrote:

>
> something looks wrong on test
>
> ======================================================================
> ERROR: test_get_info (tests.getdevinfo_tests_cygwin.TestGetInfo)
> Test that the information can be collected on this system without error
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File
> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/src/getdevinfo-1.1.0/getdevinfo/tests/getdevinfo_tests_cygwin.py",
> line 218, in test_get_info
>     cygwin.get_info()
>   File
> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
> line 101, in get_info
>     get_device_info(disk)
>   File
> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
> line 135, in get_device_info
>     cmd = subprocess.run([SMARTCTL, "-i", host_disk, "-j"],
> stdout=subprocess.PIPE,
>   File "/usr/lib/python3.8/subprocess.py", line 489, in run
>     with Popen(*popenargs, **kwargs) as process:
>   File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
>     self._execute_child(args, executable, preexec_fn, close_fds,
>   File "/usr/lib/python3.8/subprocess.py", line 1637, in _execute_child
>     self.pid = _posixsubprocess.fork_exec(
> BlockingIOError: [Errno 11] Resource temporarily unavailable
>
> ----------------------------------------------------------------------
> Ran 23 tests in 0.679s
>
> FAILED (errors=1)
> NOTE: These tests won't work correctly without administrator privileges.
>
> $ id
> uid=197609(Marco) gid=544(Administratoren)
> groups=544(Administratoren),197121(Kein)

Unfortunately I have not been able to reproduce this issue on my end
with either 32-bit or 64-bit Cygwin. What happens when you run
"/usr/sbin/smartctl.exe -i /dev/sda -j" (assuming /dev/sda is a disk
that Cygwin sees)? Note that the output may include the drive serial
number - make sure to blank it out if you post the output here.

If this is on 32-bit Cygwin, this looks like the good old fork bug to
me, seeing as you're getting "11 Resource temporarily unavailable" when
attempting to fork. I can't remember what worked to fix that for me the
last time I had it, might have been antivirus software exceptions. I
would say that maybe some packages need updating, but given you've been
releasing packages in the last few days, I highly doubt your Cygwin
install is out of date.

If the smartctl command works, could you try running the tests again please?

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 19.08.2020 13:22, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> On 19/08/2020 06:53, Marco Atzeri via Cygwin-apps wrote:
>>
>> something looks wrong on test
>>
>> ======================================================================
>> ERROR: test_get_info (tests.getdevinfo_tests_cygwin.TestGetInfo)
>> Test that the information can be collected on this system without error
>> ----------------------------------------------------------------------
>> Traceback (most recent call last):
>>    File
>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/src/getdevinfo-1.1.0/getdevinfo/tests/getdevinfo_tests_cygwin.py",
>> line 218, in test_get_info
>>      cygwin.get_info()
>>    File
>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>> line 101, in get_info
>>      get_device_info(disk)
>>    File
>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>> line 135, in get_device_info
>>      cmd = subprocess.run([SMARTCTL, "-i", host_disk, "-j"],
>> stdout=subprocess.PIPE,
>>    File "/usr/lib/python3.8/subprocess.py", line 489, in run
>>      with Popen(*popenargs, **kwargs) as process:
>>    File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
>>      self._execute_child(args, executable, preexec_fn, close_fds,
>>    File "/usr/lib/python3.8/subprocess.py", line 1637, in _execute_child
>>      self.pid = _posixsubprocess.fork_exec(
>> BlockingIOError: [Errno 11] Resource temporarily unavailable
>>
>> ----------------------------------------------------------------------
>> Ran 23 tests in 0.679s
>>
>> FAILED (errors=1)
>> NOTE: These tests won't work correctly without administrator privileges.
>>
>> $ id
>> uid=197609(Marco) gid=544(Administratoren)
>> groups=544(Administratoren),197121(Kein)
>
>
> Unfortunately I have not been able to reproduce this issue on my end
> with either 32-bit or 64-bit Cygwin. What happens when you run
> "/usr/sbin/smartctl.exe -i /dev/sda -j" (assuming /dev/sda is a disk
> that Cygwin sees)? Note that the output may include the drive serial
> number - make sure to blank it out if you post the output here.
>
> If this is on 32-bit Cygwin, this looks like the good old fork bug to
> me, seeing as you're getting "11 Resource temporarily unavailable" when
> attempting to fork. I can't remember what worked to fix that for me the
> last time I had it, might have been antivirus software exceptions. I
> would say that maybe some packages need updating, but given you've been
> releasing packages in the last few days, I highly doubt your Cygwin
> install is out of date.
>
> If the smartctl command works, could you try running the tests again please?
>
> Hamish
>

/usr/sbin/smartctl.exe -i /dev/sda -j

{
   "json_format_version": [
     1,
     0
   ],
   "smartctl": {
     "version": [
       7,
       1
     ],
     "svn_revision": "5022",
     "platform_info": "x86_64-pc-cygwin-w10-b19041",
     "build_info": "(cygwin-7.1-1)",
     "argv": [
       "smartctl",
       "-i",
       "/dev/sda",
       "-j"
     ],
     "exit_status": 0
   },
   "device": {
     "name": "/dev/sda",
     "info_name": "/dev/sda",
     "type": "ata",
     "protocol": "ATA"
   },
   "model_family": "Seagate Mobile HDD",
   "model_name": "ST1000LM035-1RK172",
   "serial_number": "WL10S143",
   "wwn": {
     "naa": 5,
     "oui": 3152,
     "id": 2907615223
   },
   "firmware_version": "RSM7",
   "user_capacity": {
     "blocks": 1953525168,
     "bytes": 1000204886016
   },
   "logical_block_size": 512,
   "physical_block_size": 4096,
   "rotation_rate": 5400,
   "form_factor": {
     "ata_value": 3,
     "name": "2.5 inches"
   },
   "in_smartctl_database": true,
   "ata_version": {
     "string": "ACS-3 T13/2161-D revision 3b",
     "major_value": 2032,
     "minor_value": 31
   },
   "sata_version": {
     "string": "SATA 3.1",
     "value": 127
   },
   "interface_speed": {
     "max": {
       "sata_value": 14,
       "string": "6.0 Gb/s",
       "units_per_second": 60,
       "bits_per_unit": 100000000
     },
     "current": {
       "sata_value": 3,
       "string": "6.0 Gb/s",
       "units_per_second": 60,
       "bits_per_unit": 100000000
     }
   },
   "local_time": {
     "time_t": 1597901572,
     "asctime": "Thu Aug 20 07:32:52 2020 CEST"
   }
}

no change on test error messages, also rebuilding from scratch in an
Admin enviroment

Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 20/08/2020 07:01, Marco Atzeri via Cygwin-apps wrote:

> On 19.08.2020 13:22, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> On 19/08/2020 06:53, Marco Atzeri via Cygwin-apps wrote:
>>>
>>> something looks wrong on test
>>>
>>> ======================================================================
>>> ERROR: test_get_info (tests.getdevinfo_tests_cygwin.TestGetInfo)
>>> Test that the information can be collected on this system without error
>>> ----------------------------------------------------------------------
>>> Traceback (most recent call last):
>>>    File
>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/src/getdevinfo-1.1.0/getdevinfo/tests/getdevinfo_tests_cygwin.py",
>>>
>>> line 218, in test_get_info
>>>      cygwin.get_info()
>>>    File
>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>>>
>>> line 101, in get_info
>>>      get_device_info(disk)
>>>    File
>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>>>
>>> line 135, in get_device_info
>>>      cmd = subprocess.run([SMARTCTL, "-i", host_disk, "-j"],
>>> stdout=subprocess.PIPE,
>>>    File "/usr/lib/python3.8/subprocess.py", line 489, in run
>>>      with Popen(*popenargs, **kwargs) as process:
>>>    File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
>>>      self._execute_child(args, executable, preexec_fn, close_fds,
>>>    File "/usr/lib/python3.8/subprocess.py", line 1637, in
>>> _execute_child
>>>      self.pid = _posixsubprocess.fork_exec(
>>> BlockingIOError: [Errno 11] Resource temporarily unavailable
>>>
>>> ----------------------------------------------------------------------
>>> Ran 23 tests in 0.679s
>>>
>>> FAILED (errors=1)
>>> NOTE: These tests won't work correctly without administrator
>>> privileges.
>>>
>>> $ id
>>> uid=197609(Marco) gid=544(Administratoren)
>>> groups=544(Administratoren),197121(Kein)
>>
>>
>> Unfortunately I have not been able to reproduce this issue on my end
>> with either 32-bit or 64-bit Cygwin. What happens when you run
>> "/usr/sbin/smartctl.exe -i /dev/sda -j" (assuming /dev/sda is a disk
>> that Cygwin sees)? Note that the output may include the drive serial
>> number - make sure to blank it out if you post the output here.
>>
>> If this is on 32-bit Cygwin, this looks like the good old fork bug to
>> me, seeing as you're getting "11 Resource temporarily unavailable" when
>> attempting to fork. I can't remember what worked to fix that for me the
>> last time I had it, might have been antivirus software exceptions. I
>> would say that maybe some packages need updating, but given you've been
>> releasing packages in the last few days, I highly doubt your Cygwin
>> install is out of date.
>>
>> If the smartctl command works, could you try running the tests again
>> please?
>>
>> Hamish
>>
>
> /usr/sbin/smartctl.exe -i /dev/sda -j
>
> {
>   "json_format_version": [
>     1,
>     0
>   ],
>   "smartctl": {
>     "version": [
>       7,
>       1
>     ],
>     "svn_revision": "5022",
>     "platform_info": "x86_64-pc-cygwin-w10-b19041",
>     "build_info": "(cygwin-7.1-1)",
>     "argv": [
>       "smartctl",
>       "-i",
>       "/dev/sda",
>       "-j"
>     ],
>     "exit_status": 0
>   },
>   "device": {
>     "name": "/dev/sda",
>     "info_name": "/dev/sda",
>     "type": "ata",
>     "protocol": "ATA"
>   },
>   "model_family": "Seagate Mobile HDD",
>   "model_name": "ST1000LM035-1RK172",
>   "serial_number": "WL10S143",
>   "wwn": {
>     "naa": 5,
>     "oui": 3152,
>     "id": 2907615223
>   },
>   "firmware_version": "RSM7",
>   "user_capacity": {
>     "blocks": 1953525168,
>     "bytes": 1000204886016
>   },
>   "logical_block_size": 512,
>   "physical_block_size": 4096,
>   "rotation_rate": 5400,
>   "form_factor": {
>     "ata_value": 3,
>     "name": "2.5 inches"
>   },
>   "in_smartctl_database": true,
>   "ata_version": {
>     "string": "ACS-3 T13/2161-D revision 3b",
>     "major_value": 2032,
>     "minor_value": 31
>   },
>   "sata_version": {
>     "string": "SATA 3.1",
>     "value": 127
>   },
>   "interface_speed": {
>     "max": {
>       "sata_value": 14,
>       "string": "6.0 Gb/s",
>       "units_per_second": 60,
>       "bits_per_unit": 100000000
>     },
>     "current": {
>       "sata_value": 3,
>       "string": "6.0 Gb/s",
>       "units_per_second": 60,
>       "bits_per_unit": 100000000
>     }
>   },
>   "local_time": {
>     "time_t": 1597901572,
>     "asctime": "Thu Aug 20 07:32:52 2020 CEST"
>   }
> }
>
> no change on test error messages, also rebuilding from scratch in an
> Admin enviroment
>
Hmm, that's unfortunate. I tried on another different machine, still
can't reproduce.

I'll get it to retry a few times if the fork error is encountered, but I
do wonder why it's happening at all. I've always seemed to have
intermittent Fork errors with Cygwin but I've never managed to pin down
exactly why it happens.

I'll update the packages with some error handling for that issue, and
some debugging code so we can see in more detail what's going wrong.

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
Okay, I have updated the packages (same location:
https://www.hamishmb.com/files/cygwin-temp/). It should now handle fork
errors better.

If it fails again, please post the full output from the test command so
I can see what version(s) of Python the tests are failing on. It might
be useful for someone else to have a go as well - would be good to know
multiple people can reproduce this issue as I have still had no luck
doing that.

Thanks for your patience,

Hamish

On 21/08/2020 16:33, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> On 20/08/2020 07:01, Marco Atzeri via Cygwin-apps wrote:
>> On 19.08.2020 13:22, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>>> On 19/08/2020 06:53, Marco Atzeri via Cygwin-apps wrote:
>>>> something looks wrong on test
>>>>
>>>> ======================================================================
>>>> ERROR: test_get_info (tests.getdevinfo_tests_cygwin.TestGetInfo)
>>>> Test that the information can be collected on this system without error
>>>> ----------------------------------------------------------------------
>>>> Traceback (most recent call last):
>>>>    File
>>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/src/getdevinfo-1.1.0/getdevinfo/tests/getdevinfo_tests_cygwin.py",
>>>>
>>>> line 218, in test_get_info
>>>>      cygwin.get_info()
>>>>    File
>>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>>>>
>>>> line 101, in get_info
>>>>      get_device_info(disk)
>>>>    File
>>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>>>>
>>>> line 135, in get_device_info
>>>>      cmd = subprocess.run([SMARTCTL, "-i", host_disk, "-j"],
>>>> stdout=subprocess.PIPE,
>>>>    File "/usr/lib/python3.8/subprocess.py", line 489, in run
>>>>      with Popen(*popenargs, **kwargs) as process:
>>>>    File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
>>>>      self._execute_child(args, executable, preexec_fn, close_fds,
>>>>    File "/usr/lib/python3.8/subprocess.py", line 1637, in
>>>> _execute_child
>>>>      self.pid = _posixsubprocess.fork_exec(
>>>> BlockingIOError: [Errno 11] Resource temporarily unavailable
>>>>
>>>> ----------------------------------------------------------------------
>>>> Ran 23 tests in 0.679s
>>>>
>>>> FAILED (errors=1)
>>>> NOTE: These tests won't work correctly without administrator
>>>> privileges.
>>>>
>>>> $ id
>>>> uid=197609(Marco) gid=544(Administratoren)
>>>> groups=544(Administratoren),197121(Kein)
>>>
>>> Unfortunately I have not been able to reproduce this issue on my end
>>> with either 32-bit or 64-bit Cygwin. What happens when you run
>>> "/usr/sbin/smartctl.exe -i /dev/sda -j" (assuming /dev/sda is a disk
>>> that Cygwin sees)? Note that the output may include the drive serial
>>> number - make sure to blank it out if you post the output here.
>>>
>>> If this is on 32-bit Cygwin, this looks like the good old fork bug to
>>> me, seeing as you're getting "11 Resource temporarily unavailable" when
>>> attempting to fork. I can't remember what worked to fix that for me the
>>> last time I had it, might have been antivirus software exceptions. I
>>> would say that maybe some packages need updating, but given you've been
>>> releasing packages in the last few days, I highly doubt your Cygwin
>>> install is out of date.
>>>
>>> If the smartctl command works, could you try running the tests again
>>> please?
>>>
>>> Hamish
>>>
>> /usr/sbin/smartctl.exe -i /dev/sda -j
>>
>> {
>>   "json_format_version": [
>>     1,
>>     0
>>   ],
>>   "smartctl": {
>>     "version": [
>>       7,
>>       1
>>     ],
>>     "svn_revision": "5022",
>>     "platform_info": "x86_64-pc-cygwin-w10-b19041",
>>     "build_info": "(cygwin-7.1-1)",
>>     "argv": [
>>       "smartctl",
>>       "-i",
>>       "/dev/sda",
>>       "-j"
>>     ],
>>     "exit_status": 0
>>   },
>>   "device": {
>>     "name": "/dev/sda",
>>     "info_name": "/dev/sda",
>>     "type": "ata",
>>     "protocol": "ATA"
>>   },
>>   "model_family": "Seagate Mobile HDD",
>>   "model_name": "ST1000LM035-1RK172",
>>   "serial_number": "WL10S143",
>>   "wwn": {
>>     "naa": 5,
>>     "oui": 3152,
>>     "id": 2907615223
>>   },
>>   "firmware_version": "RSM7",
>>   "user_capacity": {
>>     "blocks": 1953525168,
>>     "bytes": 1000204886016
>>   },
>>   "logical_block_size": 512,
>>   "physical_block_size": 4096,
>>   "rotation_rate": 5400,
>>   "form_factor": {
>>     "ata_value": 3,
>>     "name": "2.5 inches"
>>   },
>>   "in_smartctl_database": true,
>>   "ata_version": {
>>     "string": "ACS-3 T13/2161-D revision 3b",
>>     "major_value": 2032,
>>     "minor_value": 31
>>   },
>>   "sata_version": {
>>     "string": "SATA 3.1",
>>     "value": 127
>>   },
>>   "interface_speed": {
>>     "max": {
>>       "sata_value": 14,
>>       "string": "6.0 Gb/s",
>>       "units_per_second": 60,
>>       "bits_per_unit": 100000000
>>     },
>>     "current": {
>>       "sata_value": 3,
>>       "string": "6.0 Gb/s",
>>       "units_per_second": 60,
>>       "bits_per_unit": 100000000
>>     }
>>   },
>>   "local_time": {
>>     "time_t": 1597901572,
>>     "asctime": "Thu Aug 20 07:32:52 2020 CEST"
>>   }
>> }
>>
>> no change on test error messages, also rebuilding from scratch in an
>> Admin enviroment
>>
> Hmm, that's unfortunate. I tried on another different machine, still
> can't reproduce.
>
> I'll get it to retry a few times if the fork error is encountered, but I
> do wonder why it's happening at all. I've always seemed to have
> intermittent Fork errors with Cygwin but I've never managed to pin down
> exactly why it happens.
>
> I'll update the packages with some error handling for that issue, and
> some debugging code so we can see in more detail what's going wrong.
>
> Hamish
>

0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
*bump* in case this has not been seen.

Hamish

On 26/08/2020 10:35, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> Okay, I have updated the packages (same location:
> https://www.hamishmb.com/files/cygwin-temp/). It should now handle fork
> errors better.
>
> If it fails again, please post the full output from the test command so
> I can see what version(s) of Python the tests are failing on. It might
> be useful for someone else to have a go as well - would be good to know
> multiple people can reproduce this issue as I have still had no luck
> doing that.
>
> Thanks for your patience,
>
> Hamish
>
> On 21/08/2020 16:33, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> On 20/08/2020 07:01, Marco Atzeri via Cygwin-apps wrote:
>>> On 19.08.2020 13:22, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>>>> On 19/08/2020 06:53, Marco Atzeri via Cygwin-apps wrote:
>>>>> something looks wrong on test
>>>>>
>>>>> ======================================================================
>>>>> ERROR: test_get_info (tests.getdevinfo_tests_cygwin.TestGetInfo)
>>>>> Test that the information can be collected on this system without error
>>>>> ----------------------------------------------------------------------
>>>>> Traceback (most recent call last):
>>>>>    File
>>>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/src/getdevinfo-1.1.0/getdevinfo/tests/getdevinfo_tests_cygwin.py",
>>>>>
>>>>> line 218, in test_get_info
>>>>>      cygwin.get_info()
>>>>>    File
>>>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>>>>>
>>>>> line 101, in get_info
>>>>>      get_device_info(disk)
>>>>>    File
>>>>> "/pub/tmp/python-getdevinfo-1.1.0-1.src/python-getdevinfo-1.1.0-1.x86_64/build/getdevinfo/cygwin.py",
>>>>>
>>>>> line 135, in get_device_info
>>>>>      cmd = subprocess.run([SMARTCTL, "-i", host_disk, "-j"],
>>>>> stdout=subprocess.PIPE,
>>>>>    File "/usr/lib/python3.8/subprocess.py", line 489, in run
>>>>>      with Popen(*popenargs, **kwargs) as process:
>>>>>    File "/usr/lib/python3.8/subprocess.py", line 854, in __init__
>>>>>      self._execute_child(args, executable, preexec_fn, close_fds,
>>>>>    File "/usr/lib/python3.8/subprocess.py", line 1637, in
>>>>> _execute_child
>>>>>      self.pid = _posixsubprocess.fork_exec(
>>>>> BlockingIOError: [Errno 11] Resource temporarily unavailable
>>>>>
>>>>> ----------------------------------------------------------------------
>>>>> Ran 23 tests in 0.679s
>>>>>
>>>>> FAILED (errors=1)
>>>>> NOTE: These tests won't work correctly without administrator
>>>>> privileges.
>>>>>
>>>>> $ id
>>>>> uid=197609(Marco) gid=544(Administratoren)
>>>>> groups=544(Administratoren),197121(Kein)
>>>> Unfortunately I have not been able to reproduce this issue on my end
>>>> with either 32-bit or 64-bit Cygwin. What happens when you run
>>>> "/usr/sbin/smartctl.exe -i /dev/sda -j" (assuming /dev/sda is a disk
>>>> that Cygwin sees)? Note that the output may include the drive serial
>>>> number - make sure to blank it out if you post the output here.
>>>>
>>>> If this is on 32-bit Cygwin, this looks like the good old fork bug to
>>>> me, seeing as you're getting "11 Resource temporarily unavailable" when
>>>> attempting to fork. I can't remember what worked to fix that for me the
>>>> last time I had it, might have been antivirus software exceptions. I
>>>> would say that maybe some packages need updating, but given you've been
>>>> releasing packages in the last few days, I highly doubt your Cygwin
>>>> install is out of date.
>>>>
>>>> If the smartctl command works, could you try running the tests again
>>>> please?
>>>>
>>>> Hamish
>>>>
>>> /usr/sbin/smartctl.exe -i /dev/sda -j
>>>
>>> {
>>>   "json_format_version": [
>>>     1,
>>>     0
>>>   ],
>>>   "smartctl": {
>>>     "version": [
>>>       7,
>>>       1
>>>     ],
>>>     "svn_revision": "5022",
>>>     "platform_info": "x86_64-pc-cygwin-w10-b19041",
>>>     "build_info": "(cygwin-7.1-1)",
>>>     "argv": [
>>>       "smartctl",
>>>       "-i",
>>>       "/dev/sda",
>>>       "-j"
>>>     ],
>>>     "exit_status": 0
>>>   },
>>>   "device": {
>>>     "name": "/dev/sda",
>>>     "info_name": "/dev/sda",
>>>     "type": "ata",
>>>     "protocol": "ATA"
>>>   },
>>>   "model_family": "Seagate Mobile HDD",
>>>   "model_name": "ST1000LM035-1RK172",
>>>   "serial_number": "WL10S143",
>>>   "wwn": {
>>>     "naa": 5,
>>>     "oui": 3152,
>>>     "id": 2907615223
>>>   },
>>>   "firmware_version": "RSM7",
>>>   "user_capacity": {
>>>     "blocks": 1953525168,
>>>     "bytes": 1000204886016
>>>   },
>>>   "logical_block_size": 512,
>>>   "physical_block_size": 4096,
>>>   "rotation_rate": 5400,
>>>   "form_factor": {
>>>     "ata_value": 3,
>>>     "name": "2.5 inches"
>>>   },
>>>   "in_smartctl_database": true,
>>>   "ata_version": {
>>>     "string": "ACS-3 T13/2161-D revision 3b",
>>>     "major_value": 2032,
>>>     "minor_value": 31
>>>   },
>>>   "sata_version": {
>>>     "string": "SATA 3.1",
>>>     "value": 127
>>>   },
>>>   "interface_speed": {
>>>     "max": {
>>>       "sata_value": 14,
>>>       "string": "6.0 Gb/s",
>>>       "units_per_second": 60,
>>>       "bits_per_unit": 100000000
>>>     },
>>>     "current": {
>>>       "sata_value": 3,
>>>       "string": "6.0 Gb/s",
>>>       "units_per_second": 60,
>>>       "bits_per_unit": 100000000
>>>     }
>>>   },
>>>   "local_time": {
>>>     "time_t": 1597901572,
>>>     "asctime": "Thu Aug 20 07:32:52 2020 CEST"
>>>   }
>>> }
>>>
>>> no change on test error messages, also rebuilding from scratch in an
>>> Admin enviroment
>>>
>> Hmm, that's unfortunate. I tried on another different machine, still
>> can't reproduce.
>>
>> I'll get it to retry a few times if the fork error is encountered, but I
>> do wonder why it's happening at all. I've always seemed to have
>> intermittent Fork errors with Cygwin but I've never managed to pin down
>> exactly why it happens.
>>
>> I'll update the packages with some error handling for that issue, and
>> some debugging code so we can see in more detail what's going wrong.
>>
>> Hamish
>>

0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 02.09.2020 17:29, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> *bump* in case this has not been seen.
>
> Hamish
>
> On 26/08/2020 10:35, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> Okay, I have updated the packages (same location:
>> https://www.hamishmb.com/files/cygwin-temp/). It should now handle fork
>> errors better.
>>
>> If it fails again, please post the full output from the test command so
>> I can see what version(s) of Python the tests are failing on. It might
>> be useful for someone else to have a go as well - would be good to know
>> multiple people can reproduce this issue as I have still had no luck
>> doing that.
>>

Hi Hamish,

it builds fine and there are no problem on tests.

Are you missing some dependencies ?

  "On Cygwin, you need the smartmontools and blkid packages".

but they are not there:

  $ cygport python-getdevinfo.cygport deps|cat
python36-3.6.10-1
python36-bs4-4.9.1-1
python36-getdevinfo-1.1.0-1
python37-3.7.7-1
python37-bs4-4.9.1-1
python37-getdevinfo-1.1.0-1
python38-3.8.3-1
python38-bs4-4.9.1-1
python38-getdevinfo-1.1.0-1


Is already in some Linux distribution ?
Otherwise we need 5 votes from maintainers
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 02/09/2020 19:05, Marco Atzeri via Cygwin-apps wrote:

>
> Hi Hamish,
>
> it builds fine and there are no problem on tests.
>
> Are you missing some dependencies ?
>
>  "On Cygwin, you need the smartmontools and blkid packages".
>
> but they are not there:
>
>  $ cygport python-getdevinfo.cygport deps|cat
> python36-3.6.10-1
> python36-bs4-4.9.1-1
> python36-getdevinfo-1.1.0-1
> python37-3.7.7-1
> python37-bs4-4.9.1-1
> python37-getdevinfo-1.1.0-1
> python38-3.8.3-1
> python38-bs4-4.9.1-1
> python38-getdevinfo-1.1.0-1
>
>
> Is already in some Linux distribution ?
> Otherwise we need 5 votes from maintainers
Hi Marco,

Thanks for testing again :)

Ah yes, I'm not sure how I managed to do that, they should be in the
cygport file. I'll sort that out tomorrow and notify this list again.

It's not currently in the official repositories of a Linux distro no.
I'm essentially only packaging this for Cygwin because it's a dependency
for DDRescue-GUI, a GUI I wrote for GNU ddrescue. I currently release
for Linux and macOS, and there has been demand for Cygwin, hence me
getting involved with all of this in the beginning really. I would very
much appreciate it if I could get 5 votes, but if not I'll create some
kind of bundle instead. I understand that this is your project and you
may not all want my program in it :)

NB: In programs that bundle cygwin.dll, are the /dev and /cygdrive
folders visible to access devices? I'm hoping the answer is yes because
otherwise my bundle idea probably won't work.

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On Wed, Sep 2, 2020 at 10:59 PM Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>
> On 02/09/2020 19:05, Marco Atzeri via Cygwin-apps wrote:
> >
> > Hi Hamish,
> >
[cut]

> > Is already in some Linux distribution ?
> > Otherwise we need 5 votes from maintainers
>
> Hi Marco,
>
> Thanks for testing again :)
>
> Ah yes, I'm not sure how I managed to do that, they should be in the
> cygport file. I'll sort that out tomorrow and notify this list again.
>
> It's not currently in the official repositories of a Linux distro no.
> I'm essentially only packaging this for Cygwin because it's a dependency
> for DDRescue-GUI, a GUI I wrote for GNU ddrescue. I currently release
> for Linux and macOS, and there has been demand for Cygwin, hence me
> getting involved with all of this in the beginning really. I would very
> much appreciate it if I could get 5 votes, but if not I'll create some
> kind of bundle instead. I understand that this is your project and you
> may not all want my program in it :)
>
> NB: In programs that bundle cygwin.dll, are the /dev and /cygdrive
> folders visible to access devices? I'm hoping the answer is yes because
> otherwise my bundle idea probably won't work.
>
> Hamish

just to avoid misunderstandings, you have my vote, so
you need only other 3 approvals as you count as 1

Regards
Marco
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 03/09/2020 14:44, marco atzeri via Cygwin-apps wrote:
> just to avoid misunderstandings, you have my vote, so
> you need only other 3 approvals as you count as 1
>
> Regards
> Marco

Thanks for your vote Marco :) Hopefully we can get some others onboard.

Just double-checked dependencies, and both the cygport file and the
resulting hint files are correct and depend on util-linux and
smartmontools. That message is slightly wrong - blkid is in the
util-linux package - I'll fix it in the git repository now.

The "cygport <file> deps | cat" command isn't working for me either,
perhaps that's just doing automatic dependency resolution and ignoring
what I set in my cygport file? I don't see that command when I run
"cygport help" so I'm slightly fuzzy on exactly what it does.

Hamish



0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 9/3/2020 10:10 AM, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
> On 03/09/2020 14:44, marco atzeri via Cygwin-apps wrote:
>> just to avoid misunderstandings, you have my vote, so
>> you need only other 3 approvals as you count as 1
>>
>> Regards
>> Marco
>
> Thanks for your vote Marco :) Hopefully we can get some others onboard.

+1 (but you need to add smartmontools to BUILD_REQUIRES)

[...]

> The "cygport <file> deps | cat" command isn't working for me either,
> perhaps that's just doing automatic dependency resolution and ignoring
> what I set in my cygport file? I don't see that command when I run
> "cygport help" so I'm slightly fuzzy on exactly what it does.

It's not documented, but you can what it does by looking at /usr/bin/cygport.
It just does automatic dependency resolution, as you guessed.

Ken
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 03/09/2020 17:34, Ken Brown via Cygwin-apps wrote:

> On 9/3/2020 10:10 AM, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> On 03/09/2020 14:44, marco atzeri via Cygwin-apps wrote:
>>> just to avoid misunderstandings, you have my vote, so
>>> you need only other 3 approvals as you count as 1
>>>
>>> Regards
>>> Marco
>>
>> Thanks for your vote Marco :) Hopefully we can get some others onboard.
>
> +1 (but you need to add smartmontools to BUILD_REQUIRES)
>
> [...]
>
>> The "cygport <file> deps | cat" command isn't working for me either,
>> perhaps that's just doing automatic dependency resolution and ignoring
>> what I set in my cygport file? I don't see that command when I run
>> "cygport help" so I'm slightly fuzzy on exactly what it does.
>
> It's not documented, but you can what it does by looking at
> /usr/bin/cygport. It just does automatic dependency resolution, as you
> guessed.
>
> Ken
Good to know, and oh yeah, I'll do that tomorrow.

Cheers for the vote.

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 03/09/2020 18:13, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
> <cut>
> Good to know, and oh yeah, I'll do that tomorrow.
>
> Cheers for the vote.
>
> Hamish
>
I should note that this is a pure python package - can it be a noarch
package? I'm not sure how to configure the cygport file to build a
noarch package.

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 9/3/2020 1:27 PM, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> On 03/09/2020 18:13, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> <cut>
>> Good to know, and oh yeah, I'll do that tomorrow.
>>
>> Cheers for the vote.
>>
>> Hamish
>>
> I should note that this is a pure python package - can it be a noarch
> package? I'm not sure how to configure the cygport file to build a
> noarch package.

Just put the line

   ARCH=noarch

in the cygport file.

Ken
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 03/09/2020 19:50, Ken Brown via Cygwin-apps wrote:

> On 9/3/2020 1:27 PM, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> On 03/09/2020 18:13, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>>> <cut>
>>> Good to know, and oh yeah, I'll do that tomorrow.
>>>
>>> Cheers for the vote.
>>>
>>> Hamish
>>>
>> I should note that this is a pure python package - can it be a noarch
>> package? I'm not sure how to configure the cygport file to build a
>> noarch package.
>
> Just put the line
>
>   ARCH=noarch
>
> in the cygport file.
>
> Ken
Thank you, that seemed to work well.

New build dependencies added (also util-linux, binutils, cygwin, and
coreutils), and updated as a noarch package with no other changes. The
new files are in the "noarch" subdirectory at
https://www.hamishmb.com/files/cygwin-temp/.

I gave the noarch packages I built on 64-bit Cygwin a spin on 32-bit
Cygwin too, all seems fine.

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 04/09/2020 14:38, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> <cut>
> Thank you, that seemed to work well.
>
> New build dependencies added (also util-linux, binutils, cygwin, and
> coreutils), and updated as a noarch package with no other changes. The
> new files are in the "noarch" subdirectory at
> https://www.hamishmb.com/files/cygwin-temp/.
>
> I gave the noarch packages I built on 64-bit Cygwin a spin on 32-bit
> Cygwin too, all seems fine.
>
> Hamish
Just realised, that I forgot (doh!) that both GetDevInfo and
DDRescue-GUI are in the Parted Magic (www.partedmagic.com) recovery
toolkit. I work with Patrick Verner to make sure the programs work well
under that distribution as well. Does this count as a Linux Distribution?

If not, I still need two more votes, and while I know you're all
probably very busy, it'd be great if a couple more of you could take a
look at this :)

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

cygwin-apps mailing list
On 09/09/2020 11:36, Hamish McIntyre-Bhatty via Cygwin-apps wrote:

> On 04/09/2020 14:38, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> <cut>
>> Thank you, that seemed to work well.
>>
>> New build dependencies added (also util-linux, binutils, cygwin, and
>> coreutils), and updated as a noarch package with no other changes. The
>> new files are in the "noarch" subdirectory at
>> https://www.hamishmb.com/files/cygwin-temp/.
>>
>> I gave the noarch packages I built on 64-bit Cygwin a spin on 32-bit
>> Cygwin too, all seems fine.
>>
>> Hamish
> Just realised, that I forgot (doh!) that both GetDevInfo and
> DDRescue-GUI are in the Parted Magic (www.partedmagic.com) recovery
> toolkit. I work with Patrick Verner to make sure the programs work well
> under that distribution as well. Does this count as a Linux Distribution?
>
> If not, I still need two more votes, and while I know you're all
> probably very busy, it'd be great if a couple more of you could take a
> look at this :)
>
> Hamish
Hmm, who decides (and how) what counts as a Linux distro?

Parted Magic is a live disk/USB that contains a variety of system
recovery tools, but is actively maintained and has a changelog and
quality assurance etc so I suppose it could be included in the
definition of a Linux distro?

Either way, could anyone provide some insight as to whether bundling the
Cygwin DLL would allow Cygwin programs to access the virtual /dev and
/cygdrive paths? I have this all ready to be released for Windows, so
one way or another I'll need to make a bundle anyway for convenience.
It'd be great if it could make it into the official repos but I first
submitted this ITP around a month ago so I don't have high hopes as of
this point.

I will be happy to work on improving device detection/discovery in
Cygwin regardless, as and when time permits.

Hamish


0x87B761FE07F548D6.asc (3K) Download Attachment
signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ITP] python-getdevinfo

Achim Gratz
Hamish McIntyre-Bhatty via Cygwin-apps writes:
> Hmm, who decides (and how) what counts as a Linux distro?

Something that is capable of and has actually done a license review.

> Either way, could anyone provide some insight as to whether bundling the
> Cygwin DLL would allow Cygwin programs to access the virtual /dev and
> /cygdrive paths? I have this all ready to be released for Windows, so
> one way or another I'll need to make a bundle anyway for convenience.

Yes, all the virtual fs are provided through the Cygwin DLL.

> It'd be great if it could make it into the official repos but I first
> submitted this ITP around a month ago so I don't have high hopes as of
> this point.

You still haven't explained what it would be useful for.  This
bare-metal stuff isn't something I'd usually consider doing from within
a userland compatibility layer running on Windows.


Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables
12