.. it needs to operate on a FILE* rather than FIL depending on
the build.
Note that this is comparing output to expected, not to cpython dotenv
package. Because run-tests.py starts the CPython interpreter with the
'-S' (skip site initialization) flag, pip-installed packages are
not available for import inside a test file. Instead, the exp
file is generated manually:
```
circuitpython/tests$ python3 circuitpython/dotenv_test.py > circuitpython/dotenv_test.py.exp
```
Unfortunately, the test fails on test e15:
```diff
FAILURE /home/jepler/src/circuitpython/tests/results/circuitpython_dotenv_test.py
--- /home/jepler/src/circuitpython/tests/results/circuitpython_dotenv_test.py.exp 2022-10-04 09:48:16.307703128 -0500
+++ /home/jepler/src/circuitpython/tests/results/circuitpython_dotenv_test.py.out 2022-10-04 09:48:16.307703128 -0500
@@ -14,7 +14,7 @@
line
e13 e13value
e14 None
-e15 e15value
+e15 None
e16 #
e17 def
e18 #has a hash
```
This item is a UNIX symbolic link, and (except in the unlikely case
where the symlink is to a pathname that ends with a newline character!)
doesn't contain a newline.
It appears some well-intentioned tool failed to correctly handle this
file, and added a trailing newline as though it was a text file.
Removed the -# Uncrustify formatting for file
ports/broadcom/firmware
I don't know how it came into there. I didn't put it.
It is not my intention to change anything else than this branch to make 2 changes in shared_bindings/I2CTarget module
Correction of the timeout value range needed to set the timeout to 'forever'.
The line 162 checks timeout for a value of 0 while the function definition defaults timeout to -1. In the current version of the code timeout is only checked for a value of 0 or in the 'else if' part for a value of > 0. So, values of <0 will not be taken in to account. That is the reason of my modification.
Correction in function request() doc function signature.
(after speaking with @jepler on Discord).
(@jepler: 'Circuitpython always has floats enabled')
Modification of the doc text of function request().
1) The timout parameter is a keyword-only argument; so Added '*,' in the function signature;
2) for parameter timeout an integer is expected, not a float.
I made a fresh fork of circuitpython. Using Github Desktop app, created a local clone of this fork. Then Desktop app confronted me we a change in ports\broadcom\firmware -Subproject commit .nr.... and +Subproject commit same nr-dirty.
.. the value actually needs to be enforced each time the STA or AP
is enabled, because internally there's a call to cyw43_wifi_pm with the
library's defaut power management value, not ours.
Add a getter, though it only returns our idea of what the power
management register is set to, it doesn't read out from the actual
hardware, sadly.
Dan Halbert
Dan Halbert
Neradoc
Jeff Epler
Kyle Mohr
MicroDev
Bill Sideris
Paulus H.J. Schulinck
Jonathan Myers