1.8.0/1.9.0/1.10.0-RC1 -> 1.10.0 regression: Extending protocols with metadata / datafy combined with macros/symbols

2019-02-28 Thread Alex Miller
Please file a jira with this info at https://dev.clojure.org/jira/browse/CLJ -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.com Note that posts from new members are moderated - please be patie

1.8.0/1.9.0/1.10.0-RC1 -> 1.10.0 regression: Extending protocols with metadata / datafy combined with macros/symbols

2019-02-27 Thread 'somewhat-functional-programmer' via Clojure
I finally moved one of my clojure projects from 1.10.0-RC1 to 1.10.0 and encountered some very strange behaviour. Code that had previously compiled fine under 1.8.0, 1.9.0, and 1.10.0-RC1 no longer compiled under 1.10.0 with the following error: Execution error (IllegalArgumentException) at bu

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-07-05 Thread Alex Miller
There is nothing provided or planned to do that. On Tuesday, July 5, 2016 at 8:43:31 AM UTC-5, Nikita Prokopov wrote: > > How to I make spec/fdef and regular clojure.test (not generative ones) > work together? I’d like for my :ret specs and HOF :ret specs to be checked > during regular tests. I

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-07-05 Thread Nikita Prokopov
How to I make spec/fdef and regular clojure.test (not generative ones) work together? I’d like for my :ret specs and HOF :ret specs to be checked during regular tests. Is that possible? -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-17 Thread Alex Miller
explain-out is now public in master. Further mods may be coming re testing. On Thursday, June 16, 2016 at 12:52:12 PM UTC-5, Sean Corfield wrote: > > I fear you’re missing my point. > > > > You can get close to the previous nice value with: > > > > (#’s/explain-out (:result (t/check-var #’ran

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-16 Thread Sean Corfield
I fear you’re missing my point. You can get close to the previous nice value with: (#’s/explain-out (:result (t/check-var  #’ranged-rand))) But that leverages a private function / implementation detail and doesn’t handle :failed-on very nicely: boot.user=> (#'s/explain-out (:result

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-16 Thread Alex Miller
You haven't pretty-printed it to look very nice, but I think all of the same information (and more due to shrinking) is still in the check-var output. I don't know of any plan to add what you're asking for beyond what's below. {:result {:clojure.spec/problems {[] {*:pred* *(>= (:ret %) (-

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-15 Thread Sean Corfield
Given that we now have to use clojure.spec.test to get :ret / :fn tested, we lose the “nice” exceptions explaining the conformance failure: Alpha 5: ;;=> ExceptionInfo Call to #'spec-example.core/ranged-rand did not conform to spec: ;;=> At: [:fn] val: {:args {:start 8, :end 10}, :ret 7}

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-15 Thread Alex Miller
I'm in the process of updating that part of the guide right now, should be done by end of day. On Wednesday, June 15, 2016 at 12:30:55 PM UTC-5, webber wrote: > > Maybe, the following is the same reason. > > ``` > (defn ranged-rand ;; BROKEN! > "Returns random integer in range start <= rand <

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-15 Thread webber
Maybe, the following is the same reason. ``` (defn ranged-rand ;; BROKEN! "Returns random integer in range start <= rand < end" [start end] (+ start (rand-int (- start end (s/fdef ranged-rand :args (s/and (s/cat :start integer? :end integer?) #(< (:start %) (:end %)))

Re: clojure.spec regression bug for 1.9.0-alpha6

2016-06-14 Thread Alex Miller
As noted in the alpha change list, this was an intentional change in what instrument does. Instrument is intended to be used to verify that other callers have invoked a function correctly. Checking that the function works (by verifying that :ret and :fn return valid results) should be done using

clojure.spec regression bug for 1.9.0-alpha6

2016-06-14 Thread Alan Thompson
Hi - Just noticed that the :ret function in fdef seems to be ignored in 1.9.0-alpha6: user=> (require '[clojure.spec :as s]) user=> (defn dummy [x] (if x "yes" "no")) user=> (s/fdef dummy #_=> :args (s/cat :x integer?) #_=> :ret integer?) user=> (s/instrument #'dummy) user=> (dummy 3) (pr

ANN: ClojureScript 0.0-3126, fix for minor REPL regression

2015-03-18 Thread David Nolen
PL related regression. ## 0.0-3126 ### Fixes * Need to wrap REPL -setup calls in cljs.compiler/with-core-cljs -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.com Note that posts from new m

Re: clojurescript browser repl possible regression

2012-12-05 Thread Brent Millare
Ah finally, some progress. This is very relieving, I'm not that crazy it seems. It's enough for me to fix my cljs tools for now. Thank you. On Wednesday, December 5, 2012 8:35:26 AM UTC-5, Herwig Hochleitner wrote: > > I can reproduce this. > When I replace lib/goog.jar with google-closure-librar

Re: clojurescript browser repl possible regression

2012-12-05 Thread Herwig Hochleitner
One more strange fact: When you remove third-party, but run with the released closure-library, you don't see the warning, but repl still doesn't work. Thinking of it, I have run into that before. So my theory is, that the build with the plain goog.jar fails because its deps.js refers to ../third_p

Re: clojurescript browser repl possible regression

2012-12-04 Thread Brent Millare
Strange, so 'rm .repl /out/ -r' is important but only so I see the goog.async.Deferred error consistently now. Starting from scratch: git clone git://github.com/clojure/clojurescript.git cd clojurescript ./script/bootstrap cp ~/.m2/repository/org/clojure/google-closure-library-third-party/0.0-2

Re: clojurescript browser repl possible regression

2012-12-04 Thread Herwig Hochleitner
I managed to get your example to work by copying the third-party jar into lib/ One thing I ran into: Do a `rm .repl/ out/ -r` between runs, especially when compiles have failed. -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, se

Re: clojurescript browser repl possible regression

2012-12-04 Thread Brent Millare
After adding third_party jar as a dependency to classpath (by copying google-closure-library-third-party-0.0-2029.jar to clojurescript/lib, as it should automatically be picked up by the ./script/repl script) I still get the "clojure is not defined error". The other error doesn't appear. I als

Re: clojurescript browser repl possible regression

2012-12-04 Thread Herwig Hochleitner
Am 04.12.2012 09:41 schrieb "David Nolen" : > > I believe this error is because you're not including the google closure third party jar as a dependency. Yes, this is http://dev.clojure.org/jira/browse/CLJS-418 -- You received this message because you are subscribed to the Google Groups "Clojure"

Re: clojurescript browser repl possible regression

2012-12-04 Thread David Nolen
I believe this error is because you're not including the google closure third party jar as a dependency. David On Tue, Dec 4, 2012 at 2:07 AM, Brent Millare wrote: > https://github.com/downloads/bmillare/dj.web.browser/minimal.tar.gz > > @Mimmo, I don't use cljsbuild as thats mainly for automat

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
https://github.com/downloads/bmillare/dj.web.browser/minimal.tar.gz @Mimmo, I don't use cljsbuild as thats mainly for automation that doesn't fit my use cases. Instead I am using the github clojurescript docs for this reference project. Normally, I use my own build tools. I created a minimal pr

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
Please create a minimal project that demonstrates the issue for you, then we can try to run that. On Tue, Dec 4, 2012 at 12:02 AM, Brent Millare wrote: > Still getting the same error regardless. > > 1. creating index.html (with compiled cljs code and call to connect) and > putting in clojurescri

Re: clojurescript browser repl possible regression

2012-12-03 Thread Mimmo Cosenza
Have you tried using lein-cljsbuild "0.2.9" plugin without specifying any CLJS version in the :dependencies? mimmo On Dec 4, 2012, at 1:02 AM, Brent Millare wrote: > Still getting the same error regardless. > > 1. creating index.html (with compiled cljs code and call to connect) and > puttin

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
Still getting the same error regardless. 1. creating index.html (with compiled cljs code and call to connect) and putting in clojurescript directory 2. starting up clojurescript clojure repl with ./script/repl 3. Running the clojurescript repl with: (require '[cljs.repl :as repl]) (require '[cljs

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
file:// urls don't work anymore due to changes in the Google Closure Library. You need to point your browser to http://localhost:9000/, by default it looks for index.html. David On Mon, Dec 3, 2012 at 10:05 PM, Brent Millare wrote: > At the moment, all I am doing is dragging and dropping the ht

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
At the moment, all I am doing is dragging and dropping the html file into the browser, which contains the compiled clojurescript (js) which looks like this view.html ... compiled cljs stuff ... foo.main(); ... Where foo.cljs is: (ns foo (:require [clojure.browser.repl])) (defn main [] (cloju

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
It sounds like you are trying to navigate to http://localhost:9000/replthough right? You should be navigating to http://localhost:9000/ On Mon, Dec 3, 2012 at 8:51 PM, Brent Millare wrote: > The file with the clojurescript and call to connect isn't named repl, its > named view.html. repl is the

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
The file with the clojurescript and call to connect isn't named repl, its named view.html. repl is the name of the response from the call (repl/connect "http://localhost:9000/repl";) On Monday, December 3, 2012 3:29:13 PM UTC-5, David Nolen wrote: > > Don't call your file repl.* this has special

Re: clojurescript browser repl possible regression

2012-12-03 Thread David Nolen
Don't call your file repl.* this has special meaning - it's the file that's meant to be loaded into the cross page iframe. On Monday, December 3, 2012, Brent Millare wrote: > I've already tried both ways. Creating the html file with the script tag > with the simple code you showed, and just simpl

Re: clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
I've already tried both ways. Creating the html file with the script tag with the simple code you showed, and just simply trying to connect directly with http://localhost:9000/repl. In both cases, the same error message shows. Note that the error message isn't from my source code, but rather fr

clojurescript browser repl possible regression

2012-12-03 Thread Brent Millare
Using clojurescript 3842d3f9e0d68853077117a919e93e169079 Trying to do the simplest case of a clojurescript browser repl: running clojurescript/script/repl, then Taken straight from the documentation: (require '[cljs.repl :as repl])(require '[cljs.repl.browser :as browser]) ;; require the

Re: Small line-seq regression?

2009-12-14 Thread Mark Triggs
Drew Raines writes: > Rich Hickey wrote: > >> On Mon, Dec 14, 2009 at 8:48 AM, Chouser wrote: > > [...] > >>> Your analysis and solution seem right to me.  Rich, would you >>> accept a ticket for this? >>> >> >> Yes, and could someone please check the other functions that were >> patched similar

Re: Small line-seq regression?

2009-12-14 Thread Drew Raines
Rich Hickey wrote: > On Mon, Dec 14, 2009 at 8:48 AM, Chouser wrote: [...] >> Your analysis and solution seem right to me.  Rich, would you >> accept a ticket for this? >> > > Yes, and could someone please check the other functions that were > patched similarly? I'll do it since I created the

Re: Small line-seq regression?

2009-12-14 Thread Rich Hickey
On Dec 14, 4:14 pm, Sean Devlin wrote: > Is this a 1.1 or 1.2 fix? > 1.1, thanks Rich -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.com Note that posts from new members are moderated - p

Re: Small line-seq regression?

2009-12-14 Thread Sean Devlin
Is this a 1.1 or 1.2 fix? On Dec 14, 3:05 pm, Rich Hickey wrote: > On Mon, Dec 14, 2009 at 4:11 AM, Meikel Brandmeyer wrote: > > Hi, > > > Am 14.12.2009 um 01:07 schrieb Mark Triggs: > > >>  (defn line-seq > >>    "Returns the lines of text from rdr as a lazy sequence of strings. > >>    rdr mus

Re: Small line-seq regression?

2009-12-14 Thread Rich Hickey
On Mon, Dec 14, 2009 at 4:11 AM, Meikel Brandmeyer wrote: > Hi, > > Am 14.12.2009 um 01:07 schrieb Mark Triggs: > >>  (defn line-seq >>    "Returns the lines of text from rdr as a lazy sequence of strings. >>    rdr must implement java.io.BufferedReader." >>    [#^java.io.BufferedReader rdr] >>  

Re: Small line-seq regression?

2009-12-14 Thread Meikel Brandmeyer
Hi, Am 14.12.2009 um 01:07 schrieb Mark Triggs: > (defn line-seq >"Returns the lines of text from rdr as a lazy sequence of strings. >rdr must implement java.io.BufferedReader." >[#^java.io.BufferedReader rdr] >(let [line (. rdr (readLine))] > (when line >(lazy-seq

Re: Small line-seq regression?

2009-12-14 Thread Rich Hickey
On Mon, Dec 14, 2009 at 8:48 AM, Chouser wrote: > On Sun, Dec 13, 2009 at 7:07 PM, Mark Triggs wrote: >> Hi all, >> >> I recently pulled down the latest Clojure master branch and have noticed >> a small change in line-seq's behaviour which breaks some of my code. >> The code in question uses line

Re: Small line-seq regression?

2009-12-14 Thread Chouser
On Sun, Dec 13, 2009 at 7:07 PM, Mark Triggs wrote: > Hi all, > > I recently pulled down the latest Clojure master branch and have noticed > a small change in line-seq's behaviour which breaks some of my code. > The code in question uses line-seq like this: > >  (use 'clojure.contrib.duck-streams)

Small line-seq regression?

2009-12-13 Thread Mark Triggs
Hi all, I recently pulled down the latest Clojure master branch and have noticed a small change in line-seq's behaviour which breaks some of my code. The code in question uses line-seq like this: (use 'clojure.contrib.duck-streams) (with-open [ss (java.net.ServerSocket. 4141)] (println (f

Re: regression

2009-06-05 Thread kyle smith
I have generalized the code to allow more than one independent variable. The calling convention has changed: instead of separate x and y seqs, there is one "xylist". Each element of xylist is a tuple containing the dependent variable *first*, and then any independent variables. I haven't had ti

Re: regression

2009-05-27 Thread Daniel Lyons
On May 26, 2009, at 10:56 PM, kyle smith wrote: > Thanks for the feedback Daniel, I've incorporated your ideas and re- > uploaded. I'm not sure where you're seeing mutable data structures. I'm hallucinating, that's where. :) > Anyhow, I now only call eval once each time scorer is called, whic

Re: regression

2009-05-26 Thread kyle smith
Thanks for the feedback Daniel, I've incorporated your ideas and re- uploaded. I'm not sure where you're seeing mutable data structures. Anyhow, I now only call eval once each time scorer is called, which is massively faster. This has allowed for additional testing points, and now I get a perfec

Re: regression

2009-05-26 Thread Daniel Lyons
On May 26, 2009, at 2:59 PM, kyle smith wrote: > > I have written code that will randomly guess the mathematical form of > a list of xy ordered pairs. My code and a sample run are in guess- > check.clj in the files section. The final sum of squares is > fantastic! This code is just for fun, b

regression

2009-05-26 Thread kyle smith
I have written code that will randomly guess the mathematical form of a list of xy ordered pairs. My code and a sample run are in guess- check.clj in the files section. The final sum of squares is fantastic! This code is just for fun, but I would appreciate some feedback. --~--~-~--~---

Regression tests for contrib consideration

2009-01-26 Thread Shawn Hoover
fit this into Stuart Halloway's recent test_clojure build target) agents: handle OOM in agent r1198 sequences: pmap regression; - vectors equal other seqs by items equality r1208 I will post patches to issues separated by topic. I appreciate any feedback on the concepts

Re: Recent breakage (regression?) of macros

2008-10-27 Thread Phil Jordan
Hi Rich, Thanks for the quick reply! I'm probably just being slow here, for which I apologise, but some questions remain: Rich Hickey wrote: > On Oct 26, 3:49 pm, Phil Jordan <[EMAIL PROTECTED]> wrote: >> First of all, is the breakage intentional? > > Yes. You can no longer embed unreadable obj

Re: Recent breakage (regression?) of macros

2008-10-26 Thread Rich Hickey
On Oct 26, 3:49 pm, Phil Jordan <[EMAIL PROTECTED]> wrote: > Hi, > > I'm having some trouble with existing code after a recent svn update. I > think it must have happened sometime in the last 2 or so weeks, but it > might be slightly longer. > > The problematic code can be reduced to this: > > u

Recent breakage (regression?) of macros

2008-10-26 Thread Phil Jordan
Hi, I'm having some trouble with existing code after a recent svn update. I think it must have happened sometime in the last 2 or so weeks, but it might be slightly longer. The problematic code can be reduced to this: user=> (def mylocals ['A (fn [] "foo")]) #=(var user/mylocals) user=> (defma