soot-oss / SootUp

A new version of Soot with a completely overhauled architecture
https://soot-oss.github.io/SootUp/
GNU Lesser General Public License v2.1
586 stars 77 forks source link

Build fail, some files are skipped #526

Closed ox1234 closed 1 year ago

ox1234 commented 1 year ago

[WARNING] skip failed file : Could not extract header on file /home/flight/projects/java/SootUp/sootup.java.bytecode/src/main/java/sootup/java/bytecode/inputlocation/BytecodeClassLoadingOptions.java for reason copyright [Copyright (C) 2019-2020 Christian Brüggemann] is not valid org.codehaus.mojo.license.header.InvalideFileHeaderException: Could not extract header on file /home/flight/projects/java/SootUp/sootup.java.bytecode/src/main/java/sootup/java/bytecode/inputlocation/BytecodeClassLoadingOptions.java for reason copyright [Copyright (C) 2019-2020 Christian Brüggemann] is not valid

ox1234 commented 1 year ago

I use the document to install sootup. But in the final jar it missing many classes, such as JavaSootMethod, etc

ox1234 commented 1 year ago

Here is the build log: build.txt

ox1234 commented 1 year ago

only use this way, it can be import in project:

<dependencies>
        <!-- https://mvnrepository.com/artifact/commons-cli/commons-cli -->
        <dependency>
            <groupId>commons-cli</groupId>
            <artifactId>commons-cli</artifactId>
            <version>1.5.0</version>
        </dependency>
        <!-- https://mvnrepository.com/artifact/org.apache.maven/maven-model -->
        <dependency>
            <groupId>org.apache.maven</groupId>
            <artifactId>maven-model</artifactId>
            <version>3.8.5</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.core</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.java.core</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.java.core</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>

        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.callgraph</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.analysis</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.java.sourcecode</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.jimple.parser</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
        <dependency>
            <groupId>org.soot-oss</groupId>
            <artifactId>sootup.java.bytecode</artifactId>
            <version>1.0.0-SNAPSHOT</version>
        </dependency>
    </dependencies>
kadirayk commented 1 year ago

exactly, in docs we only show the sootup.core module as an example but you should add all the modules you need as a dependency.

ox1234 commented 1 year ago

exactly, in docs we only show the sootup.core module as an example but you should add all the modules you need as a dependency.

That's very confused. Maybe should add some note in the docs or just have a mechanism to package all modules in one jar?

kadirayk commented 1 year ago

Thanks a lot for the feedback. I updated the docs accordingly.