Nixpkgs-update/r-ryantm logs


#1

Hi.

I’d like to try posting the logs of nixpkgs-update/r-ryantm in this thread. The logs show which packages nixpkgs-update is failing to automatically update, and reviewing them could indicate where manual intervention is needed.

Here are the current logs:

2017-08-17 log of nixpkgs-update/r-ryantm run
2017-08-08 log of nixpkgs-update/r-ryantm run
2017-07-22 log of nixpkgs-update/r-ryantm run
2018-07-13 log of nixpkgs-update/r-ryantm run
2018-09-06 log of nixpkgs-update/r-ryantm run
2018-10-03 log of nixpkgs-update/r-ryantm run
2018-10-23 log of nixpkgs-update/r-ryantm run
2018-10-24 log of nixpkgs-update/r-ryantm python-only run
2018-11-08 log of nixpkgs-update/r-ryantm run

I will post subsequent runs as replies.

Ryan


#2

Hi, I would love to help. Could you give some hints about what needs to be done with these packages ? And how he issues can be solved ?


#3

I don’t think there is a general solution why a package failed to build.
I wonder how we can visualize this information better.
One approach could be to create branches for the each package update,
where the commit messages pings the maintainer and includes the last 500th lines of build log or so.
Then those maintainer could pull the branch and maybe fix the build.
For many packages I can already tell from the error log what went wrong.


#4

I’m excited you would love to help! There are a bunch of different ways the nixpkgs-update can fail; some of them require manual intervention, and some of them require improvements to nixpkgs-update. I’ll try to cover some of the failures that need manual intervention.

nixpkgs-update log error messages that require manual intervention

“nix build failed”

This error means nixpkgs-update successfully updated the src fixed-output derivation to the latest version, but nix-build failed. Either the package failed to build before, or the update caused it to fail. This could mean one of its dependencies is broken, a new dependency needs to be added, or something about the build process needs to be updated. The nixpkgs-update log contains the last few lines of the build output, which could be useful in seeing what went wrong, but you probably should try manually updating the package and running nix-build to see the full context.

“Could not prefetch new version URL”

Either the information from Repology.org is wrong, or upstream changed how the src is fetched. Try to find out where the new src version is obtained from and see if it builds.

“More than one fetcher in”

It refuses to update any derivation in a file that contains more than one fetcher derivation, because it isn’t smart enough to update these. These packages need to be updated manually. If you want nixpkgs-update to update these derivations in the future, try to break out the derivations into separate files, so there is only one fetcher in each file.

“nix-env -q failed to find package name with old version”

It can’t find the derivation file for the package, so it needs to be updated manually until nixpkgs-update is smart enough to find the derivation file.

Conclusion

There are more errors than this, but these ones should keep a bunch of people busy! If you have questions about other error messages, please ask. I’d be glad to help explain more.


#5

Thanks for the precise information.
Will try to give a hand. For example, in the below case, r-ryantm just needs to be told that the 2 in davfs2 is not linked to the version number. There could be a davfs2 v0.4 as well as a v3.6, right ?

2018-11-10T11:21:29 davfs2 1.5.3 -> 1.5.4
2018-11-10T11:21:37 Version in attr path davfs2 not compatible with 1.5.4
2018-11-10T11:21:37 FAIL

I am not that much into fixing packages by hand, but I would be glad to try to make r-ryantm smarter ;-).
I also like the idea of fixing download urls seems like a good idea, because it is a medium-term fix.


#6

We could add a list of packages to ignore the version attrpath compatibilty check. The general solution to this is to make all package names conform to an unambiguous convention like I propose in


#7

Thanks for pointing this out! It’s fixed and it looks like there were a bunch of false positives on that version incompatibility check.


#8

There is definitely too much in these logs. I wrote a quick script to split them at https://gist.github.com/layus/fb5ea9a5daa4126faa7cd349076f3472

There is a lot of work to do with prefetch not working (761 cases), there is a weird number of successes (217) and a expected number of failed builds (362).

I think I will start with prefetch issues.


#9

Here’s the logs from the last few days:

2018-11-17 log of nixpkgs-update/r-ryantm runs


#10

Great, I was able to run your script inside nix-shell -p parallel by renaming the log file to r-ryantm.

Please note I slightly changed the log format toward the end of this log. I put the failure message on the same line as FAIL. I haven’t spent the time to think of a good format for the log messages, if you have some ideas I’d be happy to hear them.


#11

@ryantm Are more recent logs available anywhere? I came to have a look to see if I could help at all this afternoon but couldn’t see where the logs from recent runs were.


#12

@mpickering Thanks for helping out! The logs are on my computer :). I haven’t been uploading them lately because the current code hasn’t been able to make it reliably through a complete run. That said, old logs should still be a source of work that needs doing, because most of the errors will only be fixed by human intervention (and it should be easy to see if someone already fixed it).

Here’s the latest partial run log: 20181215_ups.log


#13

@ryantm: maybe nixpkgs-update could the logs as well either in a web server with directory listing or something else simple like git with lfs-enabled. What would also help for parsing, if packages are seperated by two newlines, and compile ouput would never contain more then one newline.


#14

20181216_ups.log


#15

20181226_ups.log


#16

Happy new year! Here’s the 20190104 log


#17

20190106 log


#18

Is there any chance to get something like dub (https://github.com/NixOS/nixpkgs/blob/5041439421c86f933be749590d519928f8f9025c/pkgs/development/tools/build-managers/dub/default.nix) updated? I guess the main problem there is that there are multiple derivations in one file but simply splitting it up in different files isn’t going to work because the main derivation which is now at the bottom doesn’t even have a version or a fetcher.


#19

The issue with dub is not that it has multiple derivations. The issue is that the main derivation does not have a src attribute, which it checks for to make sure the changes it makes to the file actually update the sources for the main derivation. If you moved the src definition to the let block, and inherited it into the main derivation without it being used for anything, it would probably work. But it’s not so great to have the updater imposing something weird like that, so we should try to think of a better way.