From d4b497b00a558d440e30e4c43a978fea5c516f7d Mon Sep 17 00:00:00 2001 From: Gerrit User 1000147 <1000147@035e6965-6537-41bd-912c-053f3cf69326> Date: Fri, 12 Nov 2021 10:48:53 +0000 Subject: [PATCH] Update patch set 2 Patch Set 2: Code-Review+1 (1 comment) Patch-set: 2 Reviewer: Gerrit User 1000147 <1000147@035e6965-6537-41bd-912c-053f3cf69326> Label: Code-Review=+1 --- 30a609f03e3c1e89749c105492d36e05fd9918df | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/30a609f03e3c1e89749c105492d36e05fd9918df b/30a609f03e3c1e89749c105492d36e05fd9918df index 251680c..2e41dc0 100644 --- a/30a609f03e3c1e89749c105492d36e05fd9918df +++ b/30a609f03e3c1e89749c105492d36e05fd9918df @@ -22,6 +22,30 @@ "revId": "30a609f03e3c1e89749c105492d36e05fd9918df", "serverId": "035e6965-6537-41bd-912c-053f3cf69326", "unresolved": true + }, + { + "key": { + "uuid": "43602fb5_fc3b95d9", + "filename": "firmware/src/common/build_eeprom.py", + "patchSetId": 2 + }, + "lineNbr": 87, + "author": { + "id": 1000147 + }, + "writtenOn": "2021-11-12T10:48:53Z", + "side": 1, + "message": "looks correct according to https://stackoverflow.com/a/403426\n\nalso I\u0027ve verified that the generated files are the same in both versions.\n\n $ git checkout master\n $ autoreconf -fi\n $ ./configure\n $ make\n $ cp firmware/src/usrp2/burn-usrp* /tmp/\n $ make distclean\n $ git review -d 19675\n $ autoreconf -fi\n $ ./configure\n $ make\n $ diff -s /tmp/burn-usrp2-eeprom firmware/src/usrp2/burn-usrp2-eeprom\n Files /tmp/burn-usrp2-eeprom and firmware/src/usrp2/burn-usrp2-eeprom are identical\n $ diff -s /tmp/burn-usrp4-eeprom firmware/src/usrp2/burn-usrp4-eeprom\n Files /tmp/burn-usrp4-eeprom and firmware/src/usrp2/burn-usrp4-eeprom are identical", + "parentUuid": "015df098_dc49d469", + "range": { + "startLine": 87, + "startChar": 12, + "endLine": 87, + "endChar": 32 + }, + "revId": "30a609f03e3c1e89749c105492d36e05fd9918df", + "serverId": "035e6965-6537-41bd-912c-053f3cf69326", + "unresolved": false } ] } \ No newline at end of file