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.

Sign up for GitHub

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

Jump to bottom

cmd/snap-update-ns: fix mount rules for font sharing #4136

Merged
merged 6 commits into from
Nov 3, 2017
Merged

cmd/snap-update-ns: fix mount rules for font sharing #4136

merged 6 commits into from
Nov 3, 2017

Commits on Nov 2, 2017

  1. cmd/snap-update-ns: fix mount rules for font sharing 

    The mount rules should refer to directories, not files (the traling
slash is relevant).

Signed-off-by: Zygmunt Krynicki <me@zygoon.pl>
    @zyga
    zyga committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    19671ee View commit details
    Browse the repository at this point in the history

  2. Adding test for desktop interface

    @sergiocazzolato @zyga
    sergiocazzolato authored and zyga committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    a4804c6 View commit details
    Browse the repository at this point in the history

  3. tests: check font sharing in the desktop interface 

    Signed-off-by: Zygmunt Krynicki <me@zygoon.pl>
    @zyga
    zyga committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    e39528a View commit details
    Browse the repository at this point in the history

  4. Merge remote-tracking branch 'upstream/master' into fix/font-sharing

    @mvo5
    mvo5 committed Nov 2, 2017
    Configuration menu
    Copy the full SHA
    43d3fba View commit details
    Browse the repository at this point in the history

Commits on Nov 3, 2017

  1. Merge remote-tracking branch 'upstream/master' into HEAD

    @mvo5
    mvo5 committed Nov 3, 2017
    Configuration menu
    Copy the full SHA
    7d959de View commit details
    Browse the repository at this point in the history

  2. cmd/snap-update-ns: fix collection of changes made 

    This patch fixes a nasty bug that results in wrong list of made changes.
This is caused by how the for / range syntax operates in golang.

Before this patch the list of changes made contains N copies of the
first change. This seems to be caused by golang using a hidden temporary
variable that gets overwritten but has the same address. We add the
address of that variable to the list of changes.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
    @zyga
    zyga committed Nov 3, 2017
    Configuration menu
    Copy the full SHA
    dd810cd View commit details
    Browse the repository at this point in the history