rubyjs / mini_racer

Minimal embedded v8
MIT License
594 stars 93 forks source link

Remove unneeded files from release build #281

Closed tisba closed 1 year ago

tisba commented 1 year ago

I noticed that we include .github/ and bin/ when building. And other things :)

I think we can simplify this, by only including what we need, which is lib/**/* and ext/**/*. Additionally we probably want to continue to pack README, license, changelog and CoC.

before (released 0.6.4)

.
├── CHANGELOG
├── CODE_OF_CONDUCT.md
├── Dockerfile
├── Gemfile
├── LICENSE.txt
├── README.md
├── Rakefile
├── bin
│   ├── console
│   └── setup
├── ext
│   ├── mini_racer_extension
│   │   ├── Makefile
│   │   ├── extconf.rb
│   │   └── mini_racer_extension.cc
│   └── mini_racer_loader
│       ├── Makefile
│       ├── extconf.rb
│       └── mini_racer_loader.c
├── lib
│   ├── mini_racer
│   │   ├── truffleruby.rb
│   │   └── version.rb
│   ├── mini_racer.rb
│   ├── mini_racer_extension.bundle
│   └── mini_racer_loader.bundle
└── mini_racer.gemspec

after (build from master):

.
├── CHANGELOG
├── CODE_OF_CONDUCT.md
├── LICENSE.txt
├── README.md
├── ext
│   ├── mini_racer_extension
│   │   ├── Makefile
│   │   ├── extconf.rb
│   │   └── mini_racer_extension.cc
│   └── mini_racer_loader
│       ├── Makefile
│       ├── extconf.rb
│       └── mini_racer_loader.c
└── lib
    ├── mini_racer
    │   ├── truffleruby.rb
    │   └── version.rb
    ├── mini_racer.rb
    ├── mini_racer_extension.bundle
    └── mini_racer_loader.bundle
lloeki commented 1 year ago

Looks sensible, thanks @tisba !