)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1002394,"name":"Yoann Congal","email":"yoann.congal@smile.fr"},"change_message_id":"014e395b7c88f37b7be758ddad6d1db68e9c1262","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Sofiane HAMAM \u003csofiane.hamam@smile.fr\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-03-28 12:32:06 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"autoconf: Use SOURCE_DATE_EPOCH environment variable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This package defines PKGBLDDATE as build timestamp which makes"},{"line_number":10,"context_line":"it non reproducible. Use SOURCE_DATE_EPOCH if it is found, otherwise"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"e51f5fdb_a7d1d1f6","line":7,"updated":"2025-03-28 11:50:20.000000000","message":"We need to change the title. (I missed it, sorry)\n\nHow about: \nMakefile.am: Use SOURCE_DATE_EPOCH environment variable","commit_id":"b74fd11810a8cc9a491895afb737f9ef3ea85200"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"0ec4d6c15052d7a187a22b1a6a429e8e25c5f192","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sofiane HAMAM \u003csofiane.hamam@smile.fr\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-03-28 12:32:06 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"autoconf: Use SOURCE_DATE_EPOCH environment variable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This package defines PKGBLDDATE as build timestamp which makes"},{"line_number":10,"context_line":"it non reproducible. Use SOURCE_DATE_EPOCH if it is found, otherwise"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"e1c8b17d_ba414a8e","line":7,"in_reply_to":"e51f5fdb_a7d1d1f6","updated":"2025-03-28 11:52:56.000000000","message":"Done","commit_id":"b74fd11810a8cc9a491895afb737f9ef3ea85200"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"e1396cd336bf6c0b4dfc88bbd699127963d7c53b","unresolved":true,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I26c1a00f2e8059ae31fe72a794b5962af5a84f44"},{"line_number":17,"context_line":"Co-developed-by: default avatarYoann Congal \u003cyoann.congal@smile.fr\u003e"},{"line_number":18,"context_line":"Signed-off-by: default avatarYoann Congal \u003cyoann.congal@smile.fr\u003e"},{"line_number":19,"context_line":"Signed-off-by: Sofiane HAMAM \u003csofiane.hamam@smile.fr\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dfa2986a_6307d142","line":18,"range":{"start_line":18,"start_character":15,"end_line":18,"end_character":42},"updated":"2025-03-28 17:33:31.000000000","message":"Interesting, how that happened?","commit_id":"db213c7dea3cd3ec3cb594f8243ed0c29af5b3af"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"ea70bc92cbc2055d26f7931753ef21e5f1788e50","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I26c1a00f2e8059ae31fe72a794b5962af5a84f44"},{"line_number":17,"context_line":"Co-developed-by: default avatarYoann Congal \u003cyoann.congal@smile.fr\u003e"},{"line_number":18,"context_line":"Signed-off-by: default avatarYoann Congal \u003cyoann.congal@smile.fr\u003e"},{"line_number":19,"context_line":"Signed-off-by: Sofiane HAMAM \u003csofiane.hamam@smile.fr\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ebf9f1a8_99898be2","line":18,"range":{"start_line":18,"start_character":15,"end_line":18,"end_character":42},"in_reply_to":"3e9ca9e1_13d6c9e9","updated":"2025-03-28 18:20:48.000000000","message":"Done","commit_id":"db213c7dea3cd3ec3cb594f8243ed0c29af5b3af"},{"author":{"_account_id":1002394,"name":"Yoann Congal","email":"yoann.congal@smile.fr"},"change_message_id":"d97d7c1ecb767c056ea3443b3b8463cf55946480","unresolved":true,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I26c1a00f2e8059ae31fe72a794b5962af5a84f44"},{"line_number":17,"context_line":"Co-developed-by: default avatarYoann Congal \u003cyoann.congal@smile.fr\u003e"},{"line_number":18,"context_line":"Signed-off-by: default avatarYoann Congal \u003cyoann.congal@smile.fr\u003e"},{"line_number":19,"context_line":"Signed-off-by: Sofiane HAMAM \u003csofiane.hamam@smile.fr\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3e9ca9e1_13d6c9e9","line":18,"range":{"start_line":18,"start_character":15,"end_line":18,"end_character":42},"in_reply_to":"dfa2986a_6307d142","updated":"2025-03-28 17:56:21.000000000","message":"Nice catch!\nThat happens when commit messages are copied from our internal review tool (a Gitlab instance which displays an avatar next to every name).","commit_id":"db213c7dea3cd3ec3cb594f8243ed0c29af5b3af"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"c5093db25805430bdc7d7494853ec4c38f647a00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f3a0430a_c41b5a75","updated":"2025-03-26 13:59:16.000000000","message":"Or probably just copy example they have for Makefile there without any mods (other than different format), it doesn\u0027t need autoconf, it works on BSD and GNU/Linux systems and reviewed by many people etc.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"cde01d45fbdcbc7fc6bea49fb8a55084952baa2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4f6ad3e7_31b84ba0","updated":"2025-03-26 13:48:06.000000000","message":"Thank you for the patch, it\u0027s certainly a good thing to have reproducible builds following best practicies https://reproducible-builds.org/docs/source-date-epoch/ (you might want to reference this URL in the commit message).\n\nPlease check my alternative implementation idea inline.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"a23eb4095c9ccc00670693f351a74439502a2764","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f533c88a_f8a562bb","updated":"2025-03-26 13:29:22.000000000","message":"thanks, reproducible build is a requirement by Linux distros too, like fedora.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"fc8623211644126ca84c2ff7a636a5c2f492abf8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4869a99a_269b4f1c","in_reply_to":"009539f9_9dcc5b14","updated":"2025-03-27 09:35:20.000000000","message":"\u003e I think it needs autoconf, it\u0027s automake, it creates Makefile adjusted to the plateform.\n\nautoconf creates `configure` script adjusted to the platform. automake creates the makefiles. I do not yet understand why you say you need autoconf changes to influence automake here.\n\n\u003e With automake, conditional variables have to be defined with AM_CONDITIONAL, otherwise you get the \"error: else without if\"\n\nDo you mean one can\u0027t use normal Makefile conditionals at all with files that are processed by automake first?\n\n\u003e @Paul, how would you feel about using this autoconf m4 macro : https://www.gnu.org/software/autoconf-archive/ax_build_date_epoch.html?\n\nHi Yoann! Not sure I like it because so far I do not yet understand why autoconf changes should be necessary at all.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"6a7f43b0873449477b0328cd75682cd44ca68804","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bdd6c98a_27366987","in_reply_to":"266e842d_4211cc98","updated":"2025-03-27 15:14:01.000000000","message":"\u003e @Paul, how would you feel about using this autoconf m4 macro : https://www.gnu.org/software/autoconf-archive/ax_build_date_epoch.html?\n\nOpenOCD build doesn\u0027t currently depend on `autoconf-archive`, would be nice to keep it that way unless really justified I guess.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"e081d1c02f795bdd21ef518f7ceddcc4252dbfe5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"266e842d_4211cc98","in_reply_to":"3907940a_519f10fd","updated":"2025-03-27 11:35:18.000000000","message":"```\ndiff --git a/src/Makefile.am b/src/Makefile.am\nindex 4d1c1a250..b6731393c 100644\n--- a/src/Makefile.am\n+++ b/src/Makefile.am\n@@ -34,7 +34,7 @@ if RELEASE\n else\n %C%_libopenocd_la_CPPFLAGS +\u003d -DRELSTR\u003d\\\"`$(top_srcdir)/guess-rev.sh $(top_srcdir)`\\\"\n %C%_libopenocd_la_CPPFLAGS +\u003d -DGITVERSION\u003d\\\"`cd $(top_srcdir) \u0026\u0026 git describe`\\\"\n-%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date +%F-%R`\\\"\n+%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date $${SOURCE_DATE_EPOCH:+-d @$${SOURCE_DATE_EPOCH}} +%F-%R`\\\"\n endif\n \n # add default CPPFLAGS\n```\njust tested this and it seemed to work as expected.\n\nI do not think automake anyhow prevents regular Make conditionals from working, the fact autoconf is using AM_CONDITIONALS to affect automake behaviour doesn\u0027t imply regular Makefile conditions can\u0027t work.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"a7b79453d3c89961903f5d784f0dba7a871d69cf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3907940a_519f10fd","in_reply_to":"4869a99a_269b4f1c","updated":"2025-03-27 10:48:57.000000000","message":"AFAIK, conditionals in automake aren\u0027t the same as make, you can\u0027t use the automake if clause (if SOMETHING ... endif) without defining SOMETHING in the autoconf file. Check configure.ac in the root directory for AM_CONDITIONALs.","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1002394,"name":"Yoann Congal","email":"yoann.congal@smile.fr"},"change_message_id":"b27da6c8c7b6d0be699dbd415435ca4ced44cf67","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"009539f9_9dcc5b14","in_reply_to":"82d93d17_f9949b76","updated":"2025-03-27 08:46:52.000000000","message":"Hi! (I work with Sofiane on this)\n@Paul, how would you feel about using this autoconf m4 macro : https://www.gnu.org/software/autoconf-archive/ax_build_date_epoch.html?","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"bfa50baeb7d3456b7a66661061935838fa297c0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ef517279_c16f9559","in_reply_to":"bdd6c98a_27366987","updated":"2025-03-27 17:16:51.000000000","message":"Ack","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"31512206fde3216f8d4d47b9f29d1091f0e92175","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82d93d17_f9949b76","in_reply_to":"f3a0430a_c41b5a75","updated":"2025-03-27 08:43:48.000000000","message":"I think it needs autoconf, it\u0027s automake, it creates Makefile adjusted to the plateform.\nWith automake, conditional variables have to be defined with AM_CONDITIONAL, otherwise you get the \"error: else without if\" (https://stackoverflow.com/questions/15590920/automake-conditional-compilation-issue) \nSee : \nhttps://www.gnu.org/software/automake/manual/html_node/Usage-of-Conditionals.html","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"8c44d790f7fb0cd6b692e23a63fa307b4d102c2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"74528eb3_6ea3e46e","updated":"2025-03-27 12:20:09.000000000","message":"And yes, I see now why automake is problematic here: it neither allows to pass makefile snippets as is nor it allows to have conditionals on environment variables. Well, at least that\u0027s clear now. But why my shell-only variant can be considered worse than going via the autoconf?","commit_id":"9676c3178cde5ec1e29739f7e7e0bf7f61f30e57"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"0725d033a584fc7f634c1eaf727d549ffac5c54e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1c1a31b2_dd157b47","updated":"2025-03-27 08:38:41.000000000","message":"Hello,\nI see you updated commit message but not the patch itself, and didn\u0027t add comments in reply to mine. Probably they\u0027re saved as draft and you haven\u0027t posted yet? For that in Gerrit you need to press REPLY button on top.","commit_id":"9676c3178cde5ec1e29739f7e7e0bf7f61f30e57"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"613c931472babeec3370b7adfe02038b33238d76","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"75983768_1f804d28","in_reply_to":"1c1a31b2_dd157b47","updated":"2025-03-27 08:44:47.000000000","message":"I left my comments in draft and I didn\u0027t notice.","commit_id":"9676c3178cde5ec1e29739f7e7e0bf7f61f30e57"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"820b18fc167a89021e0df6a485b8afc30d26ac46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3ef64e6a_7c176d96","in_reply_to":"74528eb3_6ea3e46e","updated":"2025-03-27 17:15:10.000000000","message":"It is not worse, I misinterpreted the result as wrong.\nNow that you answered, I found out that I misinterpreted automake doc. As you said, :+ is expanded by shell, and I was expecting automake to do it, so I misinterpreted both the doc and the logs I get out of $(info ...).\n\nPlease check MR #8621 as I pushed to another remote branch and it created another MR instead of overwriting this one (old habits)\nDo i need to abandon it and resubmit ?","commit_id":"9676c3178cde5ec1e29739f7e7e0bf7f61f30e57"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"bfa50baeb7d3456b7a66661061935838fa297c0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"91185a41_03fabc46","in_reply_to":"75983768_1f804d28","updated":"2025-03-27 17:16:51.000000000","message":"Ack","commit_id":"9676c3178cde5ec1e29739f7e7e0bf7f61f30e57"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"aa8c40c903a50bb9af60c108d622e4a40f23f65e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b4b28366_a9a9b28a","updated":"2025-03-28 12:04:52.000000000","message":"I like this, tested the command on OpenBSD and it worked as expected.\nThank you!","commit_id":"db213c7dea3cd3ec3cb594f8243ed0c29af5b3af"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"ca20e5f13e092916e1d4f39b58624dfe44af42ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b37973c5_bd422c11","updated":"2025-03-28 13:12:47.000000000","message":"Tested full build too, it does the right thing on OpenBSD.","commit_id":"db213c7dea3cd3ec3cb594f8243ed0c29af5b3af"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"7e5d3f56ce5dab80a31cb51715923dfae1d1a67f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"dbfd88dc_1bed40e8","updated":"2025-03-29 10:01:21.000000000","message":"Thanks!","commit_id":"931680d6ffb27195bf06d38ef1e0e0de83dcde00"}],"src/Makefile.am":[{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"cde01d45fbdcbc7fc6bea49fb8a55084952baa2a","unresolved":true,"context_lines":[{"line_number":35,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DRELSTR\u003d\\\"`$(top_srcdir)/guess-rev.sh $(top_srcdir)`\\\""},{"line_number":36,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DGITVERSION\u003d\\\"`cd $(top_srcdir) \u0026\u0026 git describe`\\\""},{"line_number":37,"context_line":"if HAVE_SOURCE_DATE_EPOCH"},{"line_number":38,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date -d @$(SOURCE_DATE_EPOCH) +%F-%R`\\\""},{"line_number":39,"context_line":"else"},{"line_number":40,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date +%F-%R`\\\""},{"line_number":41,"context_line":"endif"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"a97c1724_1c43df74","line":38,"range":{"start_line":38,"start_character":51,"end_line":38,"end_character":76},"updated":"2025-03-26 13:48:06.000000000","message":"I wonder if you can spare the autoconf detection altogether and this automake conditional and instead use `${SOURCE_DATE_EPOCH:+-d @${SOURCE_DATE_EPOCH}}` to let the shell expand it when set?","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"bfa50baeb7d3456b7a66661061935838fa297c0e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DRELSTR\u003d\\\"`$(top_srcdir)/guess-rev.sh $(top_srcdir)`\\\""},{"line_number":36,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DGITVERSION\u003d\\\"`cd $(top_srcdir) \u0026\u0026 git describe`\\\""},{"line_number":37,"context_line":"if HAVE_SOURCE_DATE_EPOCH"},{"line_number":38,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date -d @$(SOURCE_DATE_EPOCH) +%F-%R`\\\""},{"line_number":39,"context_line":"else"},{"line_number":40,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date +%F-%R`\\\""},{"line_number":41,"context_line":"endif"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"4967b2d8_9f09a60f","line":38,"range":{"start_line":38,"start_character":51,"end_line":38,"end_character":76},"in_reply_to":"12f7689f_50b935e1","updated":"2025-03-27 17:16:51.000000000","message":"Ack","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1002393,"name":"Sofiane HAMAM","email":"sofiane.hamam@smile.fr","username":"sofianehamam"},"change_message_id":"31512206fde3216f8d4d47b9f29d1091f0e92175","unresolved":true,"context_lines":[{"line_number":35,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DRELSTR\u003d\\\"`$(top_srcdir)/guess-rev.sh $(top_srcdir)`\\\""},{"line_number":36,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DGITVERSION\u003d\\\"`cd $(top_srcdir) \u0026\u0026 git describe`\\\""},{"line_number":37,"context_line":"if HAVE_SOURCE_DATE_EPOCH"},{"line_number":38,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date -d @$(SOURCE_DATE_EPOCH) +%F-%R`\\\""},{"line_number":39,"context_line":"else"},{"line_number":40,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date +%F-%R`\\\""},{"line_number":41,"context_line":"endif"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"edd9dedc_f033575c","line":38,"range":{"start_line":38,"start_character":51,"end_line":38,"end_character":76},"in_reply_to":"a97c1724_1c43df74","updated":"2025-03-27 08:43:48.000000000","message":"Not sure this works for all shells\nedit : I tried it and automake does not expand it, it\u0027s always empty !","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"},{"author":{"_account_id":1000160,"name":"Paul Fertser","email":"fercerpav@gmail.com","username":"pfertser"},"change_message_id":"fc8623211644126ca84c2ff7a636a5c2f492abf8","unresolved":true,"context_lines":[{"line_number":35,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DRELSTR\u003d\\\"`$(top_srcdir)/guess-rev.sh $(top_srcdir)`\\\""},{"line_number":36,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DGITVERSION\u003d\\\"`cd $(top_srcdir) \u0026\u0026 git describe`\\\""},{"line_number":37,"context_line":"if HAVE_SOURCE_DATE_EPOCH"},{"line_number":38,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date -d @$(SOURCE_DATE_EPOCH) +%F-%R`\\\""},{"line_number":39,"context_line":"else"},{"line_number":40,"context_line":"%C%_libopenocd_la_CPPFLAGS +\u003d -DPKGBLDDATE\u003d\\\"`date +%F-%R`\\\""},{"line_number":41,"context_line":"endif"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"12f7689f_50b935e1","line":38,"range":{"start_line":38,"start_character":51,"end_line":38,"end_character":76},"in_reply_to":"edd9dedc_f033575c","updated":"2025-03-27 09:35:20.000000000","message":"\u003e Not sure this works for all shells\n\u003e edit : I tried it and automake does not expand it, it\u0027s always empty !\n\nThis syntax is POSIX shell. Automake isn\u0027t supposed to expand it, that\u0027s something shell should!","commit_id":"0b6e22e160e18201fd5308cf2e1c558a031fb124"}]}