summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #8656 from yuwata/random-fixesLennart Poettering2018-04-05139-744/+389
|\ | | | | use TAKE_PTR() and TAKE_FD() macros, and small fixes
| * core/device: trivial simplificationYu Watanabe2018-04-051-4/+2
| |
| * tree-wide: use TAKE_PTR() and TAKE_FD() macrosYu Watanabe2018-04-05136-736/+383
| |
| * network: fix typo in log messageYu Watanabe2018-04-051-1/+1
| |
| * meson: sort source filesYu Watanabe2018-04-051-2/+2
| |
| * bus-util: fix a meaningless assertionYu Watanabe2018-04-051-1/+1
|/
* Merge pull request #8605 from poettering/drop-in-name-fixZbigniew Jędrzejewski-Szmek2018-04-049-68/+186
|\ | | | | | | Fix validation of unit file drop-in naming in install.c.
| * bus-unit-util: one less newlineLennart Poettering2018-03-291-1/+0
| |
| * install: don't enforce that .d/ dropin files (and their symlink chain ↵Lennart Poettering2018-03-291-40/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | elements) for units must have names that qualify as unit names The names of drop-in files can be anything as long as they are suffixed in ".conf", hence don't be stricter than necessary when validating the names used in symlink chains of such drop-in files. Also, drop-in files should not be ale to change the type of unit file itself, i.e. not affect whether it is considered masked or an alias as a whole. This adds a flag SEARCH_DROPIN that is passed whenever we load a drop-in rather the main unit file, and in that case loosen checks and behaviour we otherwise enforce for the unit file itself. Specifically: 1. If SEARCH_DROPIN is passed we won't change the unit's info->type field anymore, as that field (which can be REGULAR, MASKED, SYMLINK) should not be affected by drop-ins, but only by the unit file itself. 2. If SEARCH_DROPIN is passed we will shortcut following of symlink chains, and not validate the naming of each element in the chain, since that's irrelevant for drop-ins, and only matters for the unit file itself. Or in other words, without this: 1. A symlink /etc/systemd/system/foobar.service.d/20-quux.conf → /dev/null might have caused the whole of foobar.service to be considered "masked". 2. A symlink /etc/systemd/system/foobar.service.d/20-quux.conf → /tmp/miepf might have caused the whole loading of foobar.service to fail as EINVAL, as "miepf" is not a valid unit name.
| * fs-util: add shortcut for chase_symlinks() when it is called like open(O_PATH)Lennart Poettering2018-03-291-0/+10
| | | | | | | | | | Let's optimize things, and let the kernel chase the paths if none of the features chase_symlinks() offers are actually used.
| * dissect: port code to chase_symlinks_and_open()Lennart Poettering2018-03-291-8/+4
| |
| * fs-util: add calls that combine chase_symlinks() and open()/opendir() in oneLennart Poettering2018-03-292-3/+90
| | | | | | | | | | This is useful when opening files within disk images, as we'll then take the relative root directory properly into account.
| * fd-util: introduce fd_reopen() helper for reopening an fdLennart Poettering2018-03-294-16/+24
| | | | | | | | | | | | We have the same code for this in place at various locations, let's unify that. Also, let's repurpose test-fs-util.c as a test for this new helper cal..
* | test-path: Set umask explicitly (#8616)Filipe Brandenburger2018-04-041-0/+4
| | | | | | | | | | | | | | | | | | | | Running `test-path` under an umask such as 027 fails with: Assertion '(s.st_mode & S_IRWXO) == 0004' failed at ../src/test/test-path.c:247, function test_path_makedirectory_directorymode(). Aborting. Looking at directory /tmp/test-path_makedirectory, it was indeed created with mode 0740, applying the umask to the requested 0744. Set an explicit umask for this test, to ensure reproducible results.
* | units: use `systemctl exit` to kill the user manager (#8648)Filipe Brandenburger2018-04-042-2/+1
| | | | | | | | | | | | | | | | | | | | | | Use `systemctl --user --force exit` to implement the systemd-exit user service. This removes our dependence on an external `kill` binary and the concerns about whether they recognize SIGRTMIN+n by name or what their interpretation of SIGRTMIN is. Tested: `systemctl --user start systemd-exit.service` kills the `systemd --user` instance for my user.
* | fix missed bracket of exec-personality-ppc64le.service (#8650)Feng Sun2018-04-041-1/+1
| | | | | | Signed-off-by: Neil Sun <neilsun@yunify.com>
* | Merge pull request #8624 from yuwata/fix-8371Zbigniew Jędrzejewski-Szmek2018-04-041-7/+8
|\ \ | | | | | | bash-completion: fixes for busctl
| * | bash-completion: busctl: make variables localYu Watanabe2018-03-301-4/+4
| | |
| * | bash-completion: busctl: do not suggest "-" for signatureYu Watanabe2018-03-301-1/+1
| | | | | | | | | | | | Fixes #8371.
| * | bash-completion: busctl: suggests only writable properties for set-propertyYu Watanabe2018-03-301-2/+3
| | |
* | | oss-fuzz: Fallback to `ninja-build` when available (#8641)Filipe Brandenburger2018-04-041-1/+9
| | | | | | | | | | | | | | | The ninja binary is deployed as `ninja-build` in older distros such as RHEL 7/CentOS 7. Detect that and use `ninja-build` instead of `ninja` when it's available.
* | | Merge pull request #8643 from poettering/coding-style-fixletsEvgeny Vereshchagin2018-04-042-5/+43
|\ \ \ | | | | | | | | Trivial CODING_STYLE additions
| * | | CODING_STYLE: document suggested suffixes to use in documentationLennart Poettering2018-04-031-0/+12
| | | |
| * | | CODING_STYLE: two minor additionsLennart Poettering2018-04-031-2/+4
| | | |
| * | | CODING_STYLE: mention that we ship .editconfig/.vimrc/.dir-locals.el in our repoLennart Poettering2018-04-031-3/+5
| | | |
| * | | update TODOLennart Poettering2018-04-031-0/+22
|/ / /
* | | tmpfiles: fix directory removal with force symlink (#8619)Ricardo Salveti de Araujo2018-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | symlink_atomic returns -EISDIR when the target symlink path is a directory. Fixes #7447 Signed-off-by: Ricardo Salveti <ricardo@opensourcefoundries.com>
* | | core: skip the removal of cgroups in the TEST_RUN_MINIMAL mode (#8622)Evgeny Vereshchagin2018-04-031-1/+1
| | | | | | | | | | | | | | | | | | When `systemd` is run in the TEST_RUN_MINIMAL mode, it doesn't really set up cgroups, so it shouldn't try to remove anything. Closes https://github.com/systemd/systemd/issues/8474.
* | | dissect: Don't count RPMB and boot partitions (#8609)João Paulo Rechi Vita2018-04-031-7/+20
| | | | | | | | | | | | | | | | | | | | | | | | Filter-out RPMB partitions and boot partitions from MMC devices when counting partitions enumerated by the kernel. Also factor out the now duplicated code into a separate function. This complement the previous fixes to the problem reported in https://github.com/systemd/systemd/issues/5806
* | | Merge pull request #8628 from jwrdegoede/hwdb-sensors2Lennart Poettering2018-04-031-9/+33
|\ \ \ | | | | | | | | Hwdb sensors2
| * | | hwdb: Add accelerometer orientation quirk for the Pipo W4 tabletHans de Goede2018-03-301-0/+4
| | | | | | | | | | | | | | | | Add an accelerometer orientation quirk for the Pipo W4 tablet.
| * | | hwdb: Add accelerometer orientation quirk for the Lenovo Ideapad Miix 310Hans de Goede2018-03-301-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an accelerometer orientation quirk for the Lenovo Ideapad Miix 310. Note this quirk is limited to the production batches which ship with a portrait panel, rather then with a landscape panel (recognized by the different BIOS version these 2 variants use).
| * | | hwdb: Add accelerometer orientation quirk for the MSI S100 2-in-1Hans de Goede2018-03-301-0/+6
| | | | | | | | | | | | | | | | Add an accelerometer orientation quirk for the MSI S100 2-in-1.
| * | | hwdb: Add accelerometer orientation quirk for the Yours Y8W81 tabletHans de Goede2018-03-301-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an accelerometer orientation quirk for the Yours Y8W81 8" tablet. For future reference: this tablet has the same case and mostly the same internals as the Chuwi Vi8. Both seem to be from an ODM called inet-tek. Both are labelled: "INET-I86M-REVxx" on the PCB, with the Chuwi Vi8 being REV03 (and having a ALC5640 audio codec) and the Yours Y8W81 being REV21 (and having a ALC5651 audio codec).
| * | | hwdb: Add accelerometer orientation entry for the I.T.Works TW701 tabletHans de Goede2018-03-301-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | Add accelerometer orientation entry for the I.T.Works TW701 7" windows tablet, note this is the same hardware/PCB as the Trekstor ST70416-6 for which we already have the same quirk.
| * | | hwdb: 60-sensors: Fix alphabetic sorting by vendor nameHans de Goede2018-03-301-13/+12
| |/ / | | | | | | | | | | | | The Cytrix entry was added at the end instead of sorted alphabetically by vendor name.
* | | man: machinectl: update fedora exmple URL (#8642)Radostin Stoyanov2018-04-031-4/+4
| | |
* | | coccinelle: fix typo in file name (#8640)Alexander Kurtz2018-04-021-0/+0
| | |
* | | Merge pull request #8626 from floppym/initctlLennart Poettering2018-03-313-6/+11
|\ \ \ | | | | | | | | sysvinit-2.89 compatibility fixes
| * | | systemctl: try opening /run/initctl before /dev/initctlMike Gilbert2018-03-301-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | This ensures compatability with old/new sysvinit. http://git.savannah.nongnu.org/cgit/sysvinit.git/commit/?id=80dbcf3de3c1b83aeaa713a8fe5b8d35d8649af2
| * | | units: initctl: move the fifo to /run/initctl to match sysvinitMike Gilbert2018-03-302-3/+3
| |/ / | | | | | | | | | | | | | | | The fifo location was moved in sysvinit-2.89. http://git.savannah.nongnu.org/cgit/sysvinit.git/commit/?id=80dbcf3de3c1b83aeaa713a8fe5b8d35d8649af2
* | | generate-af-list.sh: slightly generalize pattern, fix w/musl (#8629)Will Dietz2018-03-311-1/+1
|/ / | | | | | | | | | | | | | | | | | | Accept definitions to other AF_ constants, not just PF_ ones, such as: #define AF_LINUX AF_LOCAL It may not be necessary to impose any restriction on the definitions of the macros extracted, but for now keep most of that requirement but match AF_* as well.
* | timesync: on network event do not establish connection when NTP servers are ↵Yu Watanabe2018-03-291-5/+10
| | | | | | | | | | not changed (#8611) Fixes #8603.
* | sd-bus: allow description to be set for system/user busses (#8594)Zbigniew Jędrzejewski-Szmek2018-03-296-13/+49
|/ | | | | | | | | | | | sd_bus_open/sd_bus_open_system/sd_bus_open_user are convenient, but don't allow the description to be set. After they return, the bus is is already started, and sd_bus_set_description() fails with -EBUSY. It would be possible to allow sd_bus_set_description() to update the description "live", but messages are already emitted from sd_bus_open functions, so it's better to allow the description to be set in sd_bus_open/sd_bus_open_system/sd_bus_open_user. Fixes message like: Bus n/a: changing state UNSET → OPENING
* tree-wide: remove unused variables (#8612)Yu Watanabe2018-03-293-4/+2
|
* Merge pull request #8606 from dell/rename-suspend-to-hibernateZbigniew Jędrzejewski-Szmek2018-03-2919-73/+73
|\ | | | | Rename suspend-to-hibernate to suspend-then-hibernate
| * Fix description on suspend-then-hibernate units.Mario Limonciello2018-03-282-2/+2
| | | | | | | | Suspend; Idle into hibernate didn't make sense.
| * Rename suspend-to-hibernate to suspend-then-hibernateMario Limonciello2018-03-2819-71/+71
| | | | | | | | | | Per some discussion with Gnome folks, they would prefer this name as it's more descriptive of what's happening.
* | Merge pull request #8607 from poettering/trivialities2Yu Watanabe2018-03-298-20/+35
|\ \ | | | | | | tiny, trivial, unrelated fixes
| * | machine-image,mount-setup: minor coding style fixesLennart Poettering2018-03-282-9/+7
| | |