-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
xorriso: "Malformed environment variable SOURCE_DATE_EPOCH encountered" #23
Comments
Seems to have no affect on the final ISO, so I guess this message can be ignored ? |
That's a generic error message. The one with actual details will be a few lines up. |
Thank you Chris. You're right. Real issue seems to be:
|
@kaihendry, did you discover what the problem with this issue is exactly? Your fix doesn't seem to match the symptoms exactly. Apart from generating SOURCE_DATE_EPOCH for only the find command and apart from breaking building with sudo, your fix only seems to change the SOURCE_DATE_EPOCH to be generated from the chroot repo instead of the Debian-live-config repo, which I think would change the date, but the problem here seems to be that it is not set at all? Though, looking at the Dockerfile, it seems you only add the |
This reverts commit 89dcd1c. The commit didn't really solve the problem (it stopped making SOURCE_DATE_EPOCH available for all commands) and broke building using sudo by removing the SUDO variable for no apparent reason.
Previously, it would be based on the git version of this repo, which isn't quite correct. Also, when running the build using the Dockerfile, the .git directory of this repo would not be available, so no date could be determined, which might break the build. This should fix Webconverger#23 again.
Ah, which was probably setting |
Yup, I think that's what would happen. |
Sorry, I failed to figure out how to get a variable passing around a Makefile. Will test your branch when I get a chance. Thank you for looking into this! 😅 |
Wait, I just tested 9edba83 and I am getting different sha1sums for the resulting ISOs using https://github.com/Webconverger/Debian-Live-config/blob/master/build.sh ... am I missing something? |
I haven't actually confirmed reproducible builds (since I don't really care about that), but it the old |
Sorry for the delay, I |
I don't have any spare bandwidth in my free time to spend on this, alas... |
I had a look just now, and it seems there seems to be a timestamp in some rock-ridge related file that's different, diffing gives me this:
Note that this just dumps one iso for some context, the difference is In any case, I confirmed that the SOURCE_DATE_EPOCH is correctly passed to xorriso, with and without my changes, and the same reproducibility problem exists in the current master branch, so I think my changes are not introducing this problem. I'll go ahead and submit them as a PR to be merged (so at least builds as non-root work again) and if I have some leftover time, look at the reproducibility issue more closely. I also noticed once that git created pack files with different filenames (hashes), but perhaps I was doing something differently then (I could not reproduce this anymore, with or without my changes). |
Are you sure you are using the patched xorisso? |
I'm not sure, how do I tell ? |
I removed the patches and backports-from-sid in 40c7de8. At that time, there were no xorriso patches left in this repository, but it still backported xorriso from sid, so I assumed that the sid version has the necessary patches. At that time, the xorriso version (and other backported packages) in sid was the same as in stretch, so it seemed I could safely remove this backporting. The changelog for the stretch version (1.4.6-1, which I am running) has:
Which I think is the patch you are referring to? I also tried upgrading xorriso and its dependencies (libburn4, libisoburn1 and libisofs6) to the sid versions, but that did not make any difference. |
Actually, it seems that with the sid versions of these packages, I get a lot more differences in the resulting iso files (around 50). |
Isn't that expected, ie. you are missing the patches that I added to some WebConverger repo? (This was well over a year ago, sorry if my memory is hazy... plus slammed at the moment for time!) |
@lamby, it seems you removed those patches here (75ebaab), instead using the sid version, so I assumed the patches were merged. You mention 1.4.6 in the commit message, which I think is also what I was using before, so apparently that version is not completely ok yet (or perhaps some new issue popped up)? |
Quite possible. But I'm sorry but I just simply can't remember and don't have the bandwidth/time to reload/try all this out right now… |
That's fine, I'm not expecting you to :-) Just trying to get as much info in this ticket for when someone does have time to investigate further (I'm also a bit busy with other stuff right now). |
nodnod! |
Causing the binary target to fail.
The text was updated successfully, but these errors were encountered: