Re: [racket-dev] [plt] Push #27511: master branch updated
FWIW, this seems to have broken the snapshot builds. I'll add the missing dependency declarations so I can restart the northwestern snapshots. Also FWIW: WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/srfi/14) '(typed/rackunit) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/rackunit) '(typed/net/url) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/url) '(typed/net/uri-codec) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/uri-codec) '(typed/net/smtp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/smtp) '(typed/net/sendurl) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/sendurl) '(typed/net/sendmail) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/sendmail) '(typed/net/qp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/qp) '(typed/net/pop3) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/pop3) '(typed/net/nntp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/nntp) '(typed/net/mime) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/mime) '(typed/net/imap) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/imap) '(typed/net/head) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/head) '(typed/net/gifwrite) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/gifwrite) '(typed/net/ftp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/ftp) '(typed/net/dns) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/dns) '(typed/net/cookie) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/cookie) '(typed/net/cgi) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/cgi) '(typed/net/base64) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/base64) '(typed/file) Should those defmodules actually be @racketmodname[]'s or something? Robby On Mon, Sep 16, 2013 at 4:53 PM, Matthew Flatt mfl...@cs.utah.edu wrote: It's implemented, but it's not currently an option for `defmodule`. I'll look into adding the option. At Mon, 16 Sep 2013 17:49:27 -0400, Sam Tobin-Hochstadt wrote: I think what Carl suggested is preferable -- a link that only exists if the package it links to exists. Alternatively, a link that goes to the web if the package doesn't exist yet. I think the latter is planned but not yet implemented, from my recollection. Sam On Mon, Sep 16, 2013 at 5:41 PM, Robby Findler ro...@eecs.northwestern.edu wrote: So you want scribble to generate a link that can, when clicked (or maybe when the docs are built) turn into the real link or turn into please install package X? Robby On Monday, September 16, 2013, Sam Tobin-Hochstadt wrote: I think the Typed Racket docs should have pointers to how to use other libraries in a typed fashion -- it's a question that comes up a lot. Sam On Mon, Sep 16, 2013 at 5:16 PM, Robby Findler ro...@eecs.northwestern.edu wrote: Can that code not be moved into the relevant packages instead? Robby On Mon, Sep 16, 2013 at 3:21 PM, Asumu Takikawa as...@ccs.neu.edu wrote: On 2013-09-16 16:17:33 -0400, as...@racket-lang.org wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
On Sep 17, 2013 4:38 AM, Robby Findler ro...@eecs.northwestern.edu wrote: FWIW, this seems to have broken the snapshot builds. I'll add the missing dependency declarations so I can restart the northwestern snapshots. Is this a situation where we can make the builds more robust? Presumably this issue wouldn't have prevented the snapshot from working. Also FWIW: WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/srfi/14) '(typed/rackunit) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/rackunit) '(typed/net/url) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/url) '(typed/net/uri-codec) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/uri-codec) '(typed/net/smtp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/smtp) '(typed/net/sendurl) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/sendurl) '(typed/net/sendmail) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/sendmail) '(typed/net/qp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/qp) '(typed/net/pop3) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/pop3) '(typed/net/nntp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/nntp) '(typed/net/mime) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/mime) '(typed/net/imap) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/imap) '(typed/net/head) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/head) '(typed/net/gifwrite) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/gifwrite) '(typed/net/ftp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/ftp) '(typed/net/dns) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/dns) '(typed/net/cookie) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/cookie) '(typed/net/cgi) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/cgi) '(typed/net/base64) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/base64) '(typed/file) Should those defmodules actually be @racketmodname[]'s or something? Yes, they definitely should be. Robby On Mon, Sep 16, 2013 at 4:53 PM, Matthew Flatt mfl...@cs.utah.edu wrote: It's implemented, but it's not currently an option for `defmodule`. I'll look into adding the option. At Mon, 16 Sep 2013 17:49:27 -0400, Sam Tobin-Hochstadt wrote: I think what Carl suggested is preferable -- a link that only exists if the package it links to exists. Alternatively, a link that goes to the web if the package doesn't exist yet. I think the latter is planned but not yet implemented, from my recollection. Sam On Mon, Sep 16, 2013 at 5:41 PM, Robby Findler ro...@eecs.northwestern.edu wrote: So you want scribble to generate a link that can, when clicked (or maybe when the docs are built) turn into the real link or turn into please install package X? Robby On Monday, September 16, 2013, Sam Tobin-Hochstadt wrote: I think the Typed Racket docs should have pointers to how to use other libraries in a typed fashion -- it's a question that comes up a lot. Sam On Mon, Sep 16, 2013 at 5:16 PM, Robby Findler ro...@eecs.northwestern.edu wrote: Can that code not be moved into the relevant packages instead? Robby On Mon, Sep 16, 2013 at 3:21 PM, Asumu Takikawa as...@ccs.neu.edu wrote: On 2013-09-16 16:17:33 -0400, as...@racket-lang.org wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list:
Re: [racket-dev] [plt] Push #27511: master branch updated
On Tue, Sep 17, 2013 at 6:14 AM, Sam Tobin-Hochstadt sa...@cs.indiana.eduwrote: On Sep 17, 2013 4:38 AM, Robby Findler ro...@eecs.northwestern.edu wrote: FWIW, this seems to have broken the snapshot builds. I'll add the missing dependency declarations so I can restart the northwestern snapshots. Is this a situation where we can make the builds more robust? Presumably this issue wouldn't have prevented the snapshot from working. The snapshots don't currently distinguish between different ways that raco setup returns a non-zero exit code. And it isn't like we didn't know that there was a problem. Next time, probably best to just not commit something that makes raco setup unhappy but instead discuss it first. Robby _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
At Tue, 17 Sep 2013 07:03:30 -0500, Robby Findler wrote: On Tue, Sep 17, 2013 at 6:14 AM, Sam Tobin-Hochstadt sa...@cs.indiana.eduwrote: On Sep 17, 2013 4:38 AM, Robby Findler ro...@eecs.northwestern.edu wrote: FWIW, this seems to have broken the snapshot builds. I'll add the missing dependency declarations so I can restart the northwestern snapshots. Is this a situation where we can make the builds more robust? Presumably this issue wouldn't have prevented the snapshot from working. The snapshots don't currently distinguish between different ways that raco setup returns a non-zero exit code. And it isn't like we didn't know that there was a problem. Next time, probably best to just not commit something that makes raco setup unhappy but instead discuss it first. Yes, please treat missing package dependencies the same as any other build error. Mistakes will happen, but they should be corrected as soon as possible, even if it requires a stop-gap solution. In this particular case, if `raco setup` hadn't reported failure, the builds might have hobbled along; I'm not sure. The snapshot build generally relies on dependencies being accurate, and I don't think it's worth thinking hard about which inaccuracies might be survivable for a distribution build. _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
On 2013-09-17 06:15:42 -0600, Matthew Flatt wrote: Yes, please treat missing package dependencies the same as any other build error. Mistakes will happen, but they should be corrected as soon as possible, even if it requires a stop-gap solution. Whoops, sorry for breaking the snapshots. I thought undeclared dependencies were warnings and not errors. Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
On Tue, Sep 17, 2013 at 8:15 AM, Matthew Flatt mfl...@cs.utah.edu wrote: In this particular case, if `raco setup` hadn't reported failure, the builds might have hobbled along; I'm not sure. The snapshot build generally relies on dependencies being accurate, and I don't think it's worth thinking hard about which inaccuracies might be survivable for a distribution build. Can you say more about how the dependency info is used in the snapshot build process? In this case, I would think that the minimal build doesn't include either package involved in this, and the regular build includes both. Would that mean that the dependency error wouldn't break things? Or are there other uses of the dependency info in the snapshot process? Sam _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
At Tue, 17 Sep 2013 08:31:26 -0400, Sam Tobin-Hochstadt wrote: On Tue, Sep 17, 2013 at 8:15 AM, Matthew Flatt mfl...@cs.utah.edu wrote: In this particular case, if `raco setup` hadn't reported failure, the builds might have hobbled along; I'm not sure. The snapshot build generally relies on dependencies being accurate, and I don't think it's worth thinking hard about which inaccuracies might be survivable for a distribution build. Can you say more about how the dependency info is used in the snapshot build process? In this case, I would think that the minimal build doesn't include either package involved in this, and the regular build includes both. Would that mean that the dependency error wouldn't break things? I think you're correct, but.. Or are there other uses of the dependency info in the snapshot process? ... I wouldn't bet against that. I think the snapshot system is solid, but my experience over many small changes to the package system is that any invariant is used in more places than I remember; when we change or remove some assumption, then I end up fixing more places than I expect. It might be, for example, that installers would get created successfully, but installation would somehow trip over the assumption that `raco setup` won't report any errors in an initial install. Or maybe the install works, but as soon as you try to install a package, it runs into the same problem. I don't see either of those things happening in this case, but those are examples of where I may have overlooked something. _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
No problem. It was easy to tweak and re-run (and one lost day is not a big deal either). Robby On Tue, Sep 17, 2013 at 7:30 AM, Asumu Takikawa as...@ccs.neu.edu wrote: On 2013-09-17 06:15:42 -0600, Matthew Flatt wrote: Yes, please treat missing package dependencies the same as any other build error. Mistakes will happen, but they should be corrected as soon as possible, even if it requires a stop-gap solution. Whoops, sorry for breaking the snapshots. I thought undeclared dependencies were warnings and not errors. Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
On Tue, Sep 17, 2013 at 7:41 AM, Matthew Flatt mfl...@cs.utah.edu wrote: At Tue, 17 Sep 2013 08:31:26 -0400, Sam Tobin-Hochstadt wrote: On Tue, Sep 17, 2013 at 8:15 AM, Matthew Flatt mfl...@cs.utah.edu wrote: In this particular case, if `raco setup` hadn't reported failure, the builds might have hobbled along; I'm not sure. The snapshot build generally relies on dependencies being accurate, and I don't think it's worth thinking hard about which inaccuracies might be survivable for a distribution build. Can you say more about how the dependency info is used in the snapshot build process? In this case, I would think that the minimal build doesn't include either package involved in this, and the regular build includes both. Would that mean that the dependency error wouldn't break things? I think you're correct, but.. Or are there other uses of the dependency info in the snapshot process? ... I wouldn't bet against that. I think the snapshot system is solid, but my experience over many small changes to the package system is that any invariant is used in more places than I remember; when we change or remove some assumption, then I end up fixing more places than I expect. It might be, for example, that installers would get created successfully, but installation would somehow trip over the assumption that `raco setup` won't report any errors in an initial install. Or maybe the install works, but as soon as you try to install a package, it runs into the same problem. I don't see either of those things happening in this case, but those are examples of where I may have overlooked something. This reminds me a lot of how Jacob would talk while he was working on planet. :) Robby _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
On Tue, Sep 17, 2013 at 7:14 AM, Sam Tobin-Hochstadt sa...@cs.indiana.edu wrote: Also FWIW: WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/srfi/14) '(typed/rackunit) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/rackunit) '(typed/net/url) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/url) '(typed/net/uri-codec) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/uri-codec) '(typed/net/smtp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/smtp) '(typed/net/sendurl) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/sendurl) '(typed/net/sendmail) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/sendmail) '(typed/net/qp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/qp) '(typed/net/pop3) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/pop3) '(typed/net/nntp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/nntp) '(typed/net/mime) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/mime) '(typed/net/imap) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/imap) '(typed/net/head) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/head) '(typed/net/gifwrite) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/gifwrite) '(typed/net/ftp) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/ftp) '(typed/net/dns) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/dns) '(typed/net/cookie) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/cookie) '(typed/net/cgi) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/cgi) '(typed/net/base64) WARNING: collected information for key multiple times: '(exporting-libraries #f); values: '(typed/net/base64) '(typed/file) Should those defmodules actually be @racketmodname[]'s or something? Yes, they definitely should be. Thinking about this more, this is wrong -- these should all be `defmodule`, as they're not documented anywhere else. But the warning is a sign that something is going wrong, although I don't know where. What key is being referenced multiple times? Sam _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
On 2013-09-16 16:17:33 -0400, as...@racket-lang.org wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
Perhaps some macro hackery on the defmodule form to make it into defmodule-if-exists and only document that the module is typed if it is actually installed? Carl Eastlund On Mon, Sep 16, 2013 at 4:21 PM, Asumu Takikawa as...@ccs.neu.edu wrote: On 2013-09-16 16:17:33 -0400, as...@racket-lang.org wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
It's implemented, but it's not currently an option for `defmodule`. I'll look into adding the option. At Mon, 16 Sep 2013 17:49:27 -0400, Sam Tobin-Hochstadt wrote: I think what Carl suggested is preferable -- a link that only exists if the package it links to exists. Alternatively, a link that goes to the web if the package doesn't exist yet. I think the latter is planned but not yet implemented, from my recollection. Sam On Mon, Sep 16, 2013 at 5:41 PM, Robby Findler ro...@eecs.northwestern.edu wrote: So you want scribble to generate a link that can, when clicked (or maybe when the docs are built) turn into the real link or turn into please install package X? Robby On Monday, September 16, 2013, Sam Tobin-Hochstadt wrote: I think the Typed Racket docs should have pointers to how to use other libraries in a typed fashion -- it's a question that comes up a lot. Sam On Mon, Sep 16, 2013 at 5:16 PM, Robby Findler ro...@eecs.northwestern.edu wrote: Can that code not be moved into the relevant packages instead? Robby On Mon, Sep 16, 2013 at 3:21 PM, Asumu Takikawa as...@ccs.neu.edu wrote: On 2013-09-16 16:17:33 -0400, as...@racket-lang.org wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
I think the Typed Racket docs should have pointers to how to use other libraries in a typed fashion -- it's a question that comes up a lot. Sam On Mon, Sep 16, 2013 at 5:16 PM, Robby Findler ro...@eecs.northwestern.edu wrote: Can that code not be moved into the relevant packages instead? Robby On Mon, Sep 16, 2013 at 3:21 PM, Asumu Takikawa as...@ccs.neu.edu wrote: On 2013-09-16 16:17:33 -0400, as...@racket-lang.org wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev
Re: [racket-dev] [plt] Push #27511: master branch updated
So you want scribble to generate a link that can, when clicked (or maybe when the docs are built) turn into the real link or turn into please install package X? Robby On Monday, September 16, 2013, Sam Tobin-Hochstadt wrote: I think the Typed Racket docs should have pointers to how to use other libraries in a typed fashion -- it's a question that comes up a lot. Sam On Mon, Sep 16, 2013 at 5:16 PM, Robby Findler ro...@eecs.northwestern.edu javascript:; wrote: Can that code not be moved into the relevant packages instead? Robby On Mon, Sep 16, 2013 at 3:21 PM, Asumu Takikawa as...@ccs.neu.edujavascript:; wrote: On 2013-09-16 16:17:33 -0400, as...@racket-lang.org javascript:;wrote: +@defmodule[math #:no-declare #:link-target? #f] +@defmodule[plot/typed #:no-declare #:link-target? #f] These two lines cause `raco setup` to complain with a warning about undeclared dependencies. It seems silly, though, to have the TR doc package force a dependency on `math`. Can I quelch the warning somehow? (like a I don't really care if the dependency is missing flag) Cheers, Asumu _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev _ Racket Developers list: http://lists.racket-lang.org/dev