Skip to content
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

[#5226] Add buster support #6056

Merged
merged 3 commits into from
May 18, 2021
Merged

[#5226] Add buster support #6056

merged 3 commits into from
May 18, 2021

Conversation

gbp
Copy link
Member

@gbp gbp commented Jan 7, 2021

Relevant issue(s)

Fixes #5226
See https://github.com/mysociety/sysadmin/issues/867
Requires mysociety/commonlib#70

What does this do?

  1. Switches the wkhtmltox package
  2. Adds vagrant buster box

Why was this needed?

To support the Debian version

Implementation notes

These are commits from commonlib package repo changes

@gbp
Copy link
Member Author

gbp commented Jan 7, 2021

I've run the specs against a Buster Vagrant box - they all pass expect some Xapian index issues which seems to be related to vagrant disk IO, same failures on my Stretch box.

@garethrees is there anything else we need to do here to "support" Buster?

@garethrees
Copy link
Member

@garethrees is there anything else we need to do here to "support" Buster?

Looking at #5232, #5000 and #4218, only thing I can think of is adding a changelog entry 👍

@gbp
Copy link
Member Author

gbp commented Jan 8, 2021

Great. Will wait for mysociety/commonlib#70 then we can merge this

@gbp gbp force-pushed the add-buster-support branch from 2861622 to 337c202 Compare January 12, 2021 10:18
@gbp gbp force-pushed the add-buster-support branch from 5aee651 to 74afff9 Compare January 22, 2021 08:40
@gbp gbp marked this pull request as ready for review January 22, 2021 08:41
@garethrees garethrees changed the title Add buster support [#5226] Add buster support Jan 25, 2021
@garethrees
Copy link
Member

Noting we might need to check Zip download PDF generation still works https://github.com/mysociety/sysadmin/issues/867#issuecomment-766769338.

@gbp
Copy link
Member Author

gbp commented May 4, 2021

@garethrees think we need to get this merged. When building a bionic box I'm seeing:

    bionic: Installing packages from repository packages file...
    bionic: E
    bionic: :
    bionic: Unable to locate package wkhtmltopdf-static

@garethrees
Copy link
Member

garethrees commented May 4, 2021

Couple of things need to do before merge:

@garethrees
Copy link
Member

After rebasing on develop and updating commonlib to get mysociety/commonlib@e015b9a, this currently fails because of #6229.

    default: Bundle complete! 86 Gemfile dependencies, 168 gems now installed.
    default: Bundled gems are installed into `/home/vagrant/bundle/`
    default: Post-install message from gender_detector:
    default: For unicode support you'll need to also install the unicode_utils or activesupport gem
    default: Post-install message from icalendar:
    default: ActiveSupport is required for TimeWithZone support, but not required for general use.
    default: rake aborted!
    default: LoadError: /home/vagrant/bundle/ruby/2.5.0/extensions/x86_64-linux/2.5.0/mini_racer-0.4.0/mini_racer_extension.so: undefined symbol: _ZNSt7__cxx1119basic_ostringstreamIcSt11char_traitsIcESaIcEEC1Ev
    default: /home/vagrant/bundle/ruby/2.5.0/gems/mini_racer-0.4.0/lib/mini_racer.rb:11:in `load'
    default: /home/vagrant/bundle/ruby/2.5.0/gems/mini_racer-0.4.0/lib/mini_racer.rb:11:in `<top (required)>'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler/runtime.rb:81:in `require'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler/runtime.rb:81:in `block (2 levels) in require'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler/runtime.rb:76:in `each'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler/runtime.rb:76:in `block in require'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler/runtime.rb:65:in `each'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler/runtime.rb:65:in `require'
    default: /usr/share/rubygems-integration/all/gems/bundler-1.17.3/lib/bundler.rb:114:in `require'
    default: /home/vagrant/alaveteli/config/application.rb:21:in `<top (required)>'
    default: /home/vagrant/alaveteli/rakefile:4:in `require_relative'
    default: /home/vagrant/alaveteli/rakefile:4:in `<top (required)>'
    default: /home/vagrant/bundle/ruby/2.5.0/gems/rake-13.0.3/exe/rake:27:in `<top (required)>'
    default: (See full trace by running task with --trace)
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

@@ -35,6 +35,6 @@ ttf-bitstream-vera
unrtf
unzip
uuid-dev
wkhtmltopdf-static
wkhtmltox
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit should also change HTML_TO_PDF_COMMAND in config/general.yml-example

gbp added 3 commits May 18, 2021 10:49
This is now in the mySociety package repo for Stretch and Buster based
distros.

See: mysociety/sysadmin#867
@gbp gbp mentioned this pull request May 18, 2021
@gbp gbp force-pushed the add-buster-support branch from 74afff9 to aa5227b Compare May 18, 2021 11:29
@gbp gbp merged commit bc5e5e8 into develop May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Debian Buster support
2 participants