[ITP] python-getdevinfo

[ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 559 bytes --]

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




[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 724 bytes --]

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/

[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Marco Atzeri]

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)

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 2695 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Marco Atzeri]

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

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 5551 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 6261 bytes --]

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
>

[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 6561 bytes --]

*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
>>

[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Marco Atzeri]

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

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 1576 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[marco atzeri]

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

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 815 bytes --]

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



[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Ken Brown]

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

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 1024 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 341 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Ken Brown]

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

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 951 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 932 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 1894 bytes --]

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


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 2179 bytes --]

On 13/09/2020 07:13, Achim Gratz wrote:
> 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.
Seems fair.
>> 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.
Excellent, thank you :)
>
>> 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.

My apologies - I thought I had but that must be a false memory. This
module is a device information collector, and in Cygwin's case it makes
it easy for users of DDRescue-GUI to find the link between the Windows
drive letters and Cygwin paths. It also provides other information such
as make and model, but this is limited on Cygwin because some of the
more low-level device management utilities don't exist, probably due to
bits of missing API as explained to me by Corrina (at least I think it
was her). It behaves very similarly on macOS and Linux, except it just
needs to inform the user of device details instead of also relating to
drive letters/Windows names.

This is basically just a dependency for DDRescue-GUI (a GUI I wrote for
GNU ddrescue), and I had interest on getting this running easily on
Windows, so I thought Cygwin would be the way to go for better
compatibility. You can find DDRescue-GUI here:
https://www.hamishmb.com/html/downloads.php?program_name=ddrescue-gui
and the user guide is available from the nav bar if you want more
information about it.

Does that answer your question - hopefully I didn't miss the point.

Thanks,

Hamish


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Jon Turney]

On 03/09/2020 14:44, marco atzeri via Cygwin-apps wrote:
> 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

Oh, this stupid rule again :S

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

I don't see a statement here or on the website of the license this is under.

Assuming the answer to that is satisfactory:

+1

Re: [ITP] python-getdevinfo

[Brian Inglis]

On 2020-09-13 01:39, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
> On 13/09/2020 07:13, Achim Gratz wrote:
>> 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.
> Seems fair.
>>> 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.
> Excellent, thank you :)
>>
>>> 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.
> 
> My apologies - I thought I had but that must be a false memory. This
> module is a device information collector, and in Cygwin's case it makes
> it easy for users of DDRescue-GUI to find the link between the Windows
> drive letters and Cygwin paths. It also provides other information such
> as make and model, but this is limited on Cygwin because some of the
> more low-level device management utilities don't exist, probably due to
> bits of missing API as explained to me by Corrina (at least I think it
> was her). It behaves very similarly on macOS and Linux, except it just
> needs to inform the user of device details instead of also relating to
> drive letters/Windows names.
> 
> This is basically just a dependency for DDRescue-GUI (a GUI I wrote for
> GNU ddrescue), and I had interest on getting this running easily on
> Windows, so I thought Cygwin would be the way to go for better
> compatibility. You can find DDRescue-GUI here:
> https://www.hamishmb.com/html/downloads.php?program_name=ddrescue-gui
> and the user guide is available from the nav bar if you want more
> information about it.
> 
> Does that answer your question - hopefully I didn't miss the point.

Package smartmontools has access to some low level disk I/O features when run
elevated under an admin account.

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.
[Data in IEC units and prefixes, physical quantities in SI.]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 725 bytes --]

On 13/09/2020 15:57, Jon Turney wrote:
> On 03/09/2020 14:44, marco atzeri via Cygwin-apps wrote:
>> 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
>
> Oh, this stupid rule again :S
>
>> just to avoid misunderstandings, you have my vote, so
>> you need only other 3 approvals as you count as 1
>
> I don't see a statement here or on the website of the license this is
> under.
>
> Assuming the answer to that is satisfactory:
>
> +1

It is GPLv3+ - fully OSS.

Hamish


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 2689 bytes --]

On 13/09/2020 16:02, Brian Inglis wrote:
> On 2020-09-13 01:39, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>> On 13/09/2020 07:13, Achim Gratz wrote:
>>> 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.
>> Seems fair.
>>>> 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.
>> Excellent, thank you :)
>>>> 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.
>> My apologies - I thought I had but that must be a false memory. This
>> module is a device information collector, and in Cygwin's case it makes
>> it easy for users of DDRescue-GUI to find the link between the Windows
>> drive letters and Cygwin paths. It also provides other information such
>> as make and model, but this is limited on Cygwin because some of the
>> more low-level device management utilities don't exist, probably due to
>> bits of missing API as explained to me by Corrina (at least I think it
>> was her). It behaves very similarly on macOS and Linux, except it just
>> needs to inform the user of device details instead of also relating to
>> drive letters/Windows names.
>>
>> This is basically just a dependency for DDRescue-GUI (a GUI I wrote for
>> GNU ddrescue), and I had interest on getting this running easily on
>> Windows, so I thought Cygwin would be the way to go for better
>> compatibility. You can find DDRescue-GUI here:
>> https://www.hamishmb.com/html/downloads.php?program_name=ddrescue-gui
>> and the user guide is available from the nav bar if you want more
>> information about it.
>>
>> Does that answer your question - hopefully I didn't miss the point.
> Package smartmontools has access to some low level disk I/O features when run
> elevated under an admin account.

Yes, I think you (or maybe Corinna) told me this before - getdevinfo
compiles this together with other information to try and get as complete
a picture as possible. That was a useful tip for sure.

Hamish


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 2986 bytes --]

On 13/09/2020 17:08, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
> On 13/09/2020 16:02, Brian Inglis wrote:
>> On 2020-09-13 01:39, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
>>> On 13/09/2020 07:13, Achim Gratz wrote:
>>>> 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.
>>> Seems fair.
>>>>> 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.
>>> Excellent, thank you :)
>>>>> 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.
>>> My apologies - I thought I had but that must be a false memory. This
>>> module is a device information collector, and in Cygwin's case it makes
>>> it easy for users of DDRescue-GUI to find the link between the Windows
>>> drive letters and Cygwin paths. It also provides other information such
>>> as make and model, but this is limited on Cygwin because some of the
>>> more low-level device management utilities don't exist, probably due to
>>> bits of missing API as explained to me by Corrina (at least I think it
>>> was her). It behaves very similarly on macOS and Linux, except it just
>>> needs to inform the user of device details instead of also relating to
>>> drive letters/Windows names.
>>>
>>> This is basically just a dependency for DDRescue-GUI (a GUI I wrote for
>>> GNU ddrescue), and I had interest on getting this running easily on
>>> Windows, so I thought Cygwin would be the way to go for better
>>> compatibility. You can find DDRescue-GUI here:
>>> https://www.hamishmb.com/html/downloads.php?program_name=ddrescue-gui
>>> and the user guide is available from the nav bar if you want more
>>> information about it.
>>>
>>> Does that answer your question - hopefully I didn't miss the point.
>> Package smartmontools has access to some low level disk I/O features when run
>> elevated under an admin account.
> Yes, I think you (or maybe Corinna) told me this before - getdevinfo
> compiles this together with other information to try and get as complete
> a picture as possible. That was a useful tip for sure.
>
> Hamish

NB: If GPLv2 is required instead of GPLv3, I will happily license it as
GPLv2 for Cygwin.

I imagine any properly open source license such as GPLv3 is okay though?

Hamish


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [ITP] python-getdevinfo

[Hamish McIntyre-Bhatty]

[-- Attachment #1.1.1: Type: text/plain, Size: 451 bytes --]

On 13/09/2020 17:07, Hamish McIntyre-Bhatty via Cygwin-apps wrote:
> On 13/09/2020 15:57, Jon Turney wrote:
>> I don't see a statement here or on the website of the license this is
>> under.
>>
>> Assuming the answer to that is satisfactory:
>>
>> +1
> It is GPLv3+ - fully OSS.
>
> Hamish

Jon Turney,

Seeing as this is GPLv3+, do I have your vote?

Achim Gratz,

Did the explanation I provided answer your questions?

Hamish


[-- Attachment #1.1.2: 0x87B761FE07F548D6.asc --]
[-- Type: application/pgp-keys, Size: 3235 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]