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

Update grub to latest upstream release #22923

Closed
wants to merge 2 commits into from
Closed

Conversation

dezgeg
Copy link
Contributor

@dezgeg dezgeg commented Feb 17, 2017

There has been some upstream activity recently, let's update to the
latest upstream tag. Yes, it's -rc1 but that should be an improvement
over a year-and-a-half olf random git commit.

I guess it could be worth to get this to 17.03? cc @edolstra @globin.

I have tested this on one machine with ext4 rootfs and no separate boot. Also the installer tests I've tried pass. Would be nice to have a bit more testing, though.

There has been some upstream activity recently, let's update to the
latest upstream tag. Yes, it's -rc1 but that should be an improvement
over a year-and-a-half olf random git commit.

We also get to dump some code that's now useless, such as running
autoconf or hacks to extract prebuilt .po files from some random
release tarball.

Also the sed-patching configure is obsolete, that string doesn't exist
there anymore. It can locate an unifont.bdf from the build directory.
@mention-bot
Copy link

@dezgeg, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edolstra, @wkennington and @peti to be potential reviewers.

@dezgeg
Copy link
Contributor Author

dezgeg commented Feb 20, 2017

I gave this a try on the machine I have XFS+Grub and it worked, but didn't fix the problems as in #11650.

@grahamc
Copy link
Member

grahamc commented Mar 6, 2017

Shall we merge?

@dezgeg
Copy link
Contributor Author

dezgeg commented Mar 6, 2017

It did not fix my /boot-on-XFS problem as I had hoped, so no good reason to get into 17.03. Let's wait .

@dezgeg dezgeg mentioned this pull request Mar 30, 2017
7 tasks
@7c6f434c
Copy link
Member

@dezgeg @grahamc @fpletz I think 17.03 has happened, do we merge it now?

@orivej
Copy link
Contributor

orivej commented Nov 6, 2017

grub has been updated to 2.02 in 3317c71, but the tests here look useful.

@fpletz
Copy link
Member

fpletz commented Nov 14, 2017

Fixed the merge conflict in the test and pushed it to master.

@fpletz fpletz closed this in cbccb1e Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants