NUR icon indicating copy to clipboard operation
NUR copied to clipboard

Size limits on repos?

Open dtzWill opened this issue 6 years ago • 9 comments

I was wondering why nur-combined was so big, turns out there's a 30+ gif in someone's repo! :laughing:

Well, that'd do it. But perhaps the update checker should consider such binary assets similar to a failing evaluation? Problem is deciding how big is too big...

And sorry to name names but the repo is @Ma27's-- @Ma27 is there a good alternative approach/location for storing the demo.gif that doesn't bundle it with your expressions?

I don't think it's catastrophically bad, and mostly impacts anyone curious about that repo or using nur-combined. Questions above are more about ensuring there's not some multi-gigabyte beast in the future.

dtzWill avatar May 10 '19 20:05 dtzWill

Oh boi. Yeah, we should have a size limit.

Mic92 avatar May 11 '19 05:05 Mic92

also related to https://github.com/nix-community/NUR/pull/132

Mic92 avatar May 11 '19 05:05 Mic92

Maybe we can do a size limit per file also?

Mic92 avatar May 11 '19 05:05 Mic92

I also thought about excluding large repositories from being included in nur-combined.

Mic92 avatar May 11 '19 05:05 Mic92

for my repo i do a subdirectory-filter with git filter-branch to avoid exporting all my systems config,modules,libs and tests ( https://git.shackspace.de/krebs/stockholm/blob/master/.gitlab-ci.yml#L15 ).

makefu avatar May 11 '19 17:05 makefu

And sorry to name names but the repo is @Ma27's-- @Ma27 is there a good alternative approach/location for storing the demo.gif that doesn't bundle it with your expressions?

That's perfectly fine, in the end you're absolutely right, that was a pretty bad idea :/

I have to admit that I currently don't use the NUR mechanism as I figured some time ago that simply using nix.nixPath and import <nixexprs> { /* ... */ } suits better for my workflow. As it isn't the first time I got pinged due to some problem with my repository I guess that it's the best for everyone if I remove my repo from NUR for now.

Ma27 avatar May 11 '19 18:05 Ma27

I suppose we would need a history rewrite to undo this.

Mic92 avatar May 12 '19 21:05 Mic92

Looks like the build has been broken since #134. Is there a fix planned?

yurrriq avatar May 15 '19 14:05 yurrriq

:tada: https://travis-ci.com/nix-community/NUR/builds/112879185

yurrriq avatar May 23 '19 07:05 yurrriq