카드 리더기 기준 파일(cljc)에 대한 지원 추가(#1827)

묘사

Clojure 1.7.0은 카드 리더기 조건문에 대한 지원을 증가시킵니다.
http://dev.clojure.org/display/design/Reader+ 조건문
이 패치는leiningen에 독자 조건 지원을 추가했습니다. 특히
컴파일 (aot 및 오래된 파일 포함) 과 테스트.
이 패치를 병합하기 전에 Raynes/bultitude cljc 파일의 이름 공간을 해석하는 지원을 풀고 어떤 방식으로 수정해야 합니다 path-for. 이름 공간과 원본 파일 사이에 1대1로 비추는 것이 불가능하기 때문입니다.통합 요청을 작성했습니다.https://github.com/Raynes/bultitude/pull/28
이 변화를 받아들이는 데 도움이 될 수 있는 일이 있다면 알려주세요.나는 이 지원을 추가하기를 매우 갈망한다. 왜냐하면 우리의 비즈니스 프로젝트는 aot가 필요하기 때문에 이 is기능을 추가하기 전에, 우리는 리더 조건을 사용할 수 없다.

토론 #1

업데이트 버전의 Raynes/Bulitude에 대한 의존이 부족하여 생성에 실패했습니다.사용 가능한 경우 병합 요청을 업데이트합니다.

토론 #2

출시.

토론 #셋

분명히 스냅샷을 발표하지 않았습니다.

토론 #4

감사합니다!
만약 우리가 .cljc 파일을 사용하는 Leiningen 사용자들에게 이것을 보여줄 수 있다면 매우 유용할 것이다.합병하기 전에 나는 더 많은 피드백을 얻고 싶다.

토론 #5

전체 운영체제에 새 버전의lein을 설치하지 않은 상태에서 이 변경 사항을 시도할 수 있습니까?

토론 #6

@bensu예, CONTRIBUTING.md의'안내'부분을 보세요.

토론 #7

@technomancy 감사합니다!
나는 미니멀repo에서 이 패치를 시도했지만 성공하지 못했다.내가 잘못 뛰었나 봐bin/lein.
나는 leiningen와 montoux:ft-reader-conditionals의 새 사본, 그리고 테스트 환매 협의cljc-test가 같은 폴더 아래에 있다.그래서 도망갔어요.

 cd leiningen/leiningen-core
 lein bootstrap
 cd ..
 bin/lein compile

 cd ../cljc-test
 ../leiningen/bin/lein test

결과는 이gist

토론 #8

에 상세하게 설명되어 있습니다. 감사합니다. @bensu, 제가 한번 볼게요.HEAD의 기초를 다시 설정하고 필요에 따라pull 요청을 업데이트할 것입니다.

토론 #9

@bensu는 이 두 가지 점에서 모두 정확하다.이 변화는 liningen(또는 더 구체적으로 말하면 Bulitude)이 clojure 1.7의 리더를 사용하여 ns표를 불러올 것을 요구할 것이다.또한, Bulitude는read의 사용을 조정해야 합니다. :read-cond :allow 옵션을 추가해야 합니다.
Bulitute에 필요한 변경 사항을 다시 요청해서 기쁘지만, 발표할 때 Leiningen을 clojure 1.7.0으로 업그레이드하는 타당성에 대한 피드백을 받고 싶습니다.이게 가능합니까?그렇지 않으면 Bulitude와 다른 곳에서 다른 리더를 사용해야 합니다.

토론 #10

현재 이 요청은 합병이나 테스트가 준비되지 않았습니다.Leiningen이 발표될 때(또는 Clojure 1.7.0 지점일 수도 있음) Clojure 1.7.0으로 전환하는 것과 다른 리더를 사용할 가능성에 대한 피드백을 알고 싶습니다.leiningen의 다른 부분은 영향을 받을 수 있습니다. 만약 우리가 1.7로 전환하지 않으면 프로젝트에eval이 없으면, leiningen은 cljc 파일을 읽을 수 없습니다.

토론 #11

tl이 존재하는지 여부;dr는 이미 알고 있는 돌파적인 변경 사항을 지원합니까? 아니면 완전히 호환되는 업그레이드입니까?

토론 #12

나는 어떤 돌파적인 변화가 있는지 모르겠다
안녕하세요.
나는 이 패치가 급히 필요하다.이것은 아직 마스터에 병합되지 않았습니다. 그렇습니까?

토론 #13

1.7이 발표된 지 몇 주가 되었는데, 언제 마스터에 합병될 ETA가 있습니까?

토론 #14

더 많은 테스트가 필요하다면 클라라 규칙에 따라 테스트할 수 있습니다.우리는 대부분의 코드를 리더 조건과 통합할 것입니다. 나는 현재 통합 테스트를 하고 있습니다.CCW/Eclipse에서 실행할 수 있지만, 분명히, 이 패치는 설정 조건의 읽기를 지원하기 위해 Bultute를 업데이트하기 위해 끊긴pull 요청을 기다리고 있습니다. (https://github.com/Raynes/bultitude/pull/29)(@raynes).완료되면,pull 요청을 업데이트할 것입니다.이것은 liningen을 Clojure 1.7.0으로 전환하도록 요구합니다. @technomancy의 서명이 필요합니다.

토론 #15

pull 요청을 업데이트했고commit을 추가하여 Leiningen을 Clojure 1.7.0으로 업데이트했습니다.@bensu에서 관찰한 문제를 복구했습니다. Bulitude에 대한 다른 변경이 필요합니다. Bulitude가 통합되었습니다. (@raynes 감사합니다.)코드는 심사할 준비가 되어 있지만, 테스트 인원들은 Bulitutede 발표를 미루려고 할 수도 있습니다. (내일일 수도 있습니다.)
나는 이미 우리의 내부 프로젝트로 이것에 대해 테스트를 진행했는데, 이것은 cljx에서 이식된 상당히 큰 cljc 테스트를 포함한다.나는 우리의 상당히 큰 내부 프로젝트로 liningen을 Clojure 1.6.0에서 1.7.0으로 업그레이드하는 데 문제가 없지만, 이것은 틀림없이 다른 프로젝트의 추가 테스트에서 이익을 얻을 것이다.

토론 #16

만약에 알려진 돌파적인 변화가 없다면 Leiningen을 1.7.0으로 업데이트할 수 있지만 이 패치가 발표되기 전에 더 많은 사람들이 그것을 테스트하는 것이 현명하다고 생각합니다.

토론 #17

저는 이번 주말까지 완성할 것입니다.이 패치가 발표되기 전에, 나는 또 해야 할 일이 하나 있지만, 나는 이번 주에 그것을 잊어버리고 싶다.
감사합니다

토론 #18

Pull 요청이 테스트 준비가 되었습니다

토론 #19

. 저는 다음 항목에서 테스트

토론 #20

를 시도했습니다. 이 항목들은

토론 #21

테스트 파일이 있습니다.
- zelkova
- bidi's cljc branch
- 제 일은 사적인 프로젝트라 설치가 상당히 복잡해요.
다음 작업을 수행할 수 있습니다.
- montoux/ft-reader-conditionals- cljc- lein test- lein cljsbuild모든 것이 순조롭다.lein repl 내가 이 요청을 받아들이는 것을 도울 수 있는 것은 무엇입니까?lein doo 만약에 우리가 한 명 이상의 테스트 인원이 있다면 나는 더욱 좋을 것 같지만, 나는 그것을 마스터에 통합할 수 있다. 마치 우리가 실제 버전을 잘라내기 전에 더 많은 테스트 인원들이 그것을 사용하도록 시도할 수 있는 것처럼.