- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.1k
Upgrade to sbt 1 #3441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
          
     Closed
      
        
      
    
  
     Closed
                    Upgrade to sbt 1 #3441
Changes from all commits
      Commits
    
    
            Show all changes
          
          
            49 commits
          
        
        Select commit
          Hold shift + click to select a range
      
      6aafb69
              
                sbt 1.0.0-M6
              
              
                smarter 4ce83b8
              
                Upgrade to sbt 1.0.0-M6
              
              
                smarter c91171d
              
                Fix sbt bridge
              
              
                ttreyer 024f5fb
              
                Add method SymDenotation.isTopLevelClass
              
              
                ttreyer e1018d1
              
                Remove SimpleType
              
              
                ttreyer 7865186
              
                Update bridge for compatibility with Zinc 1.0.0-X15
              
              
                ttreyer 3f8a49c
              
                Fix for zinc 1.0.0-X16
              
              
                smarter 848e41b
              
                ExtractAPI: fix children of sealed classes
              
              
                smarter bfd2b0f
              
                ExtractAPI: add missing startSource callback
              
              
                smarter d3f50a4
              
                Fix various issues
              
              
                smarter f02dacb
              
                Propagate class files through GenBCode pipeline
              
              
                ttreyer d7dba97
              
                Catch FileConflictException creating class files
              
              
                ttreyer fb9c611
              
                Adapt to latest Zinc
              
              
                Duhemm aef68c2
              
                Compile only the bridge, not the whole compiler
              
              
                Duhemm 0f13174
              
                Adapt to new main class detection in Zinc 1
              
              
                Duhemm 323b451
              
                Re-enable `dist-*` projects
              
              
                Duhemm c05353a
              
                Fix `bin/dot*` scripts
              
              
                Duhemm 4c91ed2
              
                Fix `source-dependencies/abstract-type-override`
              
              
                Duhemm 2cf0abe
              
                Fix `source-dependencies/transitive-memberRef`
              
              
                Duhemm f232fbe
              
                Disable `source-dependencies/inherited-deps-java`
              
              
                Duhemm 494beb9
              
                Fix test `source-dependencies/compactify`
              
              
                Duhemm b749648
              
                Mark test `source-dependencies/binary` pending
              
              
                Duhemm dd7841d
              
                Fix test source-dependencies/export-jars
              
              
                Duhemm 98c9e8d
              
                Update test `source-dependencies/import-class`
              
              
                Duhemm 479c6c2
              
                Fix test source-dependencies/trait-member-modified
              
              
                Duhemm 240e133
              
                Fix test source-dependencies/java-analyis-serialization-error
              
              
                Duhemm 08e09b8
              
                Fix test source-dependencies/canon
              
              
                Duhemm 939dd31
              
                Fix test source-dependencies/restore-classes
              
              
                Duhemm 4ee6fd5
              
                Fix test source-dependencies/type-alias
              
              
                Duhemm 159a19e
              
                Fix test source-dependencies/typeref-only
              
              
                Duhemm 149c8b1
              
                Fix test source-dependencies/less-inter-inv-java
              
              
                Duhemm 389cdcc
              
                Fix test source-dependencies/backtick-qupted-names
              
              
                Duhemm ca77743
              
                Fix test source-dependencies/replace-test-a
              
              
                Duhemm e4f092d
              
                Fix test source-dependencies/inherited_type_params
              
              
                Duhemm ce237c7
              
                Fix test source-dependencies/same-file-used-names
              
              
                Duhemm b46a4b3
              
                Fix test source-dependencies/ext
              
              
                Duhemm 9281d1c
              
                Fix test source-dependencies/less-inter-inv
              
              
                Duhemm ff34abb
              
                Fix test sbt-dotty/example-project
              
              
                Duhemm 805eceb
              
                Fix test compilerReporter/simple
              
              
                Duhemm 4475e9e
              
                Fix test source-dependencies/value-class-underlying
              
              
                Duhemm 5b5b260
              
                Disable test source-dependencies/trait-private-var
              
              
                Duhemm df2ac08
              
                Always run scripted tests
              
              
                Duhemm d818a2c
              
                Split scripted tests in two groups
              
              
                Duhemm 2075f13
              
                Remove `Message` and `Log`
              
              
                Duhemm 39ac06b
              
                Upgrade to sbt 1.0.3
              
              
                Duhemm f35894c
              
                Change name of scala-library jar in tests
              
              
                Duhemm ef10d7c
              
                Address review comments
              
              
                Duhemm 80d36d5
              
                Port sbt/zinc#444
              
              
                Duhemm 7705124
              
                Address review comments
              
              
                Duhemm File filter
Filter by extension
Conversations
          Failed to load comments.   
        
        
          
      Loading
        
  Jump to
        
          Jump to file
        
      
      
          Failed to load files.   
        
        
          
      Loading
        
  Diff view
Diff view
There are no files selected for viewing
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
  
    
      This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
      Learn more about bidirectional Unicode characters
    
  
  
    
              
      
      Oops, something went wrong.
        
    
  
  Add this suggestion to a batch that can be applied as a single commit.
  This suggestion is invalid because no changes were made to the code.
  Suggestions cannot be applied while the pull request is closed.
  Suggestions cannot be applied while viewing a subset of changes.
  Only one suggestion per line can be applied in a batch.
  Add this suggestion to a batch that can be applied as a single commit.
  Applying suggestions on deleted lines is not supported.
  You must change the existing code in this line in order to create a valid suggestion.
  Outdated suggestions cannot be applied.
  This suggestion has been applied or marked resolved.
  Suggestions cannot be applied from pending reviews.
  Suggestions cannot be applied on multi-line comments.
  Suggestions cannot be applied while the pull request is queued to merge.
  Suggestion cannot be applied right now. Please check back later.
  
    
  
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great if we did not introduce this huge regression (3x) in the time needed to run the scripted tests.
@Duhemm says it is because the bridge is recompiled for each scripted test. @smarter do you have an idea how this could be fixed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Complain to the sbt people: sbt/sbt#3469 Maybe ask someone who has a lightbend subscription? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternatively, we could try copy-pasting the custom implementation of scripted used by zinc, it even has parallelism and fancy colors contributed by @jvican :P. https://github.com/sbt/zinc/blob/1.x/project/Scripted.scala https://github.com/sbt/zinc/tree/1.x/internal/zinc-scripted/src/test/scala/sbt/internal/inc