cmu-db / peloton

The Self-Driving Database Management System
http://pelotondb.io
Apache License 2.0
2.03k stars 623 forks source link

packages.sh: Ubuntu 18.04 LTS support, removed deprecated OSes #1390

Closed mbutrovich closed 6 years ago

mbutrovich commented 6 years ago

Tested on clean install of Ubuntu 18.04 LTS in a VM. Passed make check in Debug with ASAN on. Added package is due to a linker error (-lz) experienced during build on 18.04.

Removed code related to Red Hat, Debian, and Fedora and replaced with error messages.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.01%) to 76.975% when pulling cdc1ef5f75166db152bff54dd72e5951469e57e2 on mbutrovich:packages into fa6b48c5622becb4ae51a7357b23d42f714e86f4 on cmu-db:master.

mbutrovich commented 6 years ago

Not sure what happened to the history here when I did the suggested changes and then a rebase. That merge of master by @tli2 combined with a later rebase might have messed with it. I'll take a look in the AM and nuke it if needed. It's only 1 file to change if I make the branch again.

apavlo commented 6 years ago

@mbutrovich Probably best to just extract out your one commit and make a new branch.

mbutrovich commented 6 years ago

@apavlo I think I got it sorted out. If the commit history looks weird I can redo it, but it looks okay to me now.