@pmatilai commented on this pull request.


>  
-    /* Limit the maximum date to SOURCE_DATE_EPOCH if defined
-     * similar to the tar --clamp-mtime option
-     * https://reproducible-builds.org/specs/source-date-epoch/
-     */
-    if (srcdate && rpmExpandNumeric("%{?clamp_mtime_to_source_date_epoch}")) {
-       char *endptr;
-       errno = 0;
-       source_date_epoch = strtol(srcdate, &endptr, 10);
-       if (srcdate == endptr || *endptr || errno != 0) {
-           rpmlog(RPMLOG_ERR, _("unable to parse %s=%s\n"), 
"SOURCE_DATE_EPOCH", srcdate);
-           fl->processingFailed = 1;
+    /* backward compatibility */
+    if (!*mtime_policy_str) {
+        if (srcdate && 
rpmExpandNumeric("%{?clamp_mtime_to_source_date_epoch}")) {

Maybe this should issue a deprecation warning?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2944#pullrequestreview-1975738512
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2944/review/1975738...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to