mflatt
2017-10-28 13:57:46

DrRacket refrains from applying errortrace to anything that’s from the main “collects” or from packages that you’re not editing. There are many reasons for that rule, among them that DrRacket cannot in general write to “compiled” directories in those places.


mflatt
2017-10-28 13:59:44

It’s true that I never got around to adding it to pkgs, and I should do that. I think the “distro-build” collection is as intended, though.


samth
2017-10-28 14:25:07

If it’s in that collection, then the checklist instruction is wrong


samth
2017-10-28 14:26:44

Or maybe the instructions expect a particular current directory?


mflatt
2017-10-28 16:40:56

I don’t see any particular problem, either


mflatt
2017-10-28 16:53:48

I’ve pushed repairs, where the two expanders had different bugs, but the same underlying cause that I hadn’t considered the possibility of internal-definition contexts for 'module-begin mode. I wouldn’t be surprised if you run into further problems, especially with the old expander.


mflatt
2017-10-28 16:58:48

The checklist was wrong; now fixed


mflatt
2017-10-28 16:59:08

I didn’t notice because I do always run the test from its directory


lexi.lambda
2017-10-28 17:13:32

@mflatt Thank you for both of those fixes, though I think ultimately I can get away with neither of them, since the strategy of just using unique, global scopes for namespacing eliminates my need for splicing-syntax-parameterize. (Which almost seems disappointing, since it was a fun use-case, but this approach works a lot better.)


me1
2017-10-28 22:30:16

@me1 has joined the channel


samth
2017-10-28 23:56:50

@greg This test continues to be somewhat flaky http://drdr.racket-lang.org/43204/racket/share/pkgs/http/http/request.rkt ; would it make sense to add more retries?


greg
2017-10-29 00:00:26

@samth Although I’ve only glanced at it I think @jeapostrophe’s PR from this morning will do that: https://github.com/greghendershott/http/pull/10


greg
2017-10-29 00:01:25

has been eyeball deep in something and is a few weeks behind on PRs overall


samth
2017-10-29 00:01:32

That URL definitely responds for me


notjack
2017-10-29 00:04:11

what if the catalog test runner gave tests the ability to run docker containers? You could make a non-flaky http test suite with that


samth
2017-10-29 00:05:03

the issue here is the CI server at http://drdr.racket-lang.org\|drdr.racket-lang.org, which doesn’t have the restrictions that the pkg-build system does


samth
2017-10-29 00:05:19

but does apparently have periodic issues reaching http://yahoo.com\|yahoo.com


notjack
2017-10-29 00:05:34

bizarre


samth
2017-10-29 00:07:07

and it’s really genuinely just http://yahoo.com\|yahoo.com, as you can see at that link (click the green back button a bunch of times to see various issues)


notjack
2017-10-29 00:15:46

the drdr links aren’t working for me


notjack
2017-10-29 00:15:52

not able to connect



samth
2017-10-29 00:16:34

it works for me


notjack
2017-10-29 00:17:11

….oh I have https everywhere set to block non-https connections because I was using public wifi hotspots…


notjack
2017-10-29 00:20:06

it’s yahoo closing the tcp connection so I don’t think there’s much you can do there


samth
2017-10-29 00:20:45

yeah but it usually works so i think just retrying a few times should fix it


notjack
2017-10-29 00:21:09

How often does that test run?


samth
2017-10-29 00:21:24

about once an hour


samth
2017-10-29 00:21:37

the CI server just runs continously


samth
2017-10-29 00:22:03

(it used to be triggered by pushes but we gave up on that when we split the main repo)


notjack
2017-10-29 00:22:18

gotcha


samth
2017-10-29 00:22:59

the main http://drdr.racket-lang.org\|drdr.racket-lang.org page shows the history of all builds


notjack
2017-10-29 00:23:31

how different are drdr and the package build server?


samth
2017-10-29 00:23:50

quite


samth
2017-10-29 00:24:22

drdr is intended to be a highly comprehensive test of the main distribution


samth
2017-10-29 00:25:36

for example, it builds from source every time, runs tests that need a gui or arbitrary system access or the network, and tests every file in every package in main-distribution and main-distribution-test, and doesn’t have an overall time limit


samth
2017-10-29 00:26:01

but it doesn’t build anything that’s not in the main-distribution or needed by it


notjack
2017-10-29 00:26:54

interesting


notjack
2017-10-29 00:27:07

gui bits tested with xvfb?


notjack
2017-10-29 00:28:21

oh by the way @samth, could you add a license to samth/docker-racket-build?


samth
2017-10-29 00:29:27

done


notjack
2017-10-29 00:29:56

perfect, thanks


samth
2017-10-29 00:30:19

actually, i want to add the other license too


notjack
2017-10-29 00:31:04

other license?


samth
2017-10-29 00:32:00

apache as well


notjack
2017-10-29 00:32:05

oh, dual licensing it?


notjack
2017-10-29 00:32:22

you’re probably better off using just apache instead of using dual licenses


notjack
2017-10-29 00:32:42

licenses don’t compose as neatly as functions do unfortunately


samth
2017-10-29 00:33:13

We’re dual-licensing Racket generally as MIT and Apache, for roughly the same reasons that Rust does


samth
2017-10-29 00:33:20

so I want to do the same thing here


notjack
2017-10-29 00:33:31

oh rust does that? neat


samth
2017-10-29 00:34:24

yes


notjack
2017-10-29 00:34:34

ah, I gotcha, dual licensing means that pull request authors implicitly agree to both


samth
2017-10-29 00:34:53

yeah


samth
2017-10-29 00:35:13

that’s why I wanted to do it now before you do anything that might get you in trouble with Google Legal


samth
2017-10-29 00:37:22

oh, and it actually runs a real X server, not xvfb


notjack
2017-10-29 00:38:34

good call on license thing then - relicensing stuff that’s partially copyright google would be… not simple, probably


notjack
2017-10-29 00:38:50

as for real X server: why?


samth
2017-10-29 00:42:29

not certain, @jeapostrophe would know better


notjack
2017-10-29 00:42:42

oh and, how are snapshot and pre-release installers made?


samth
2017-10-29 00:43:01

those are built with the distro-build package


notjack
2017-10-29 00:44:36

do you want the dockerfile to only describe how to build stuff in terms of what’s at http://pre-release.racket-lang.org\|pre-release.racket-lang.org or do you want it to conceivably work in racket CI before those are uploaded anywhere?


samth
2017-10-29 00:45:39

mostly the former


samth
2017-10-29 00:46:09

the goal is to automate the process when @stamourv sends out an email telling people to do testing items that are on their checklist


samth
2017-10-29 00:46:21

notjack
2017-10-29 00:46:55

and I presume the way that’s done is people take the pre-release installer and install racket with it, then run those tests?


samth
2017-10-29 00:47:06

yes


samth
2017-10-29 00:47:43

and so this is just automating that process in a nice reproducible way


notjack
2017-10-29 00:48:00

kk


notjack
2017-10-29 00:48:19

then I think ill file some issues about stuff we could do


samth
2017-10-29 00:49:11

there’s an issue filed already for all the things that don’t work (which may be harder to fix for you), plus a couple of issues about the Docker code


samth
2017-10-29 00:49:21

based on what you suggested yesterday


notjack
2017-10-29 01:45:03

@samth filed issues for all the stuff I could think of off the top of my head