Skip to content
Snippets Groups Projects

base: Fix open-file locking

Closed base: Fix open-file locking
wip/fix-ofd-locking into master
All threads resolved!
Closed Ben Gamari requested to merge wip/fix-ofd-locking into master
All threads resolved!

The OFD locking path introduced in 3b784d44 due to #13945 (closed) appears to have never actually worked but we never noticed due to an oversight in the autoconf check. Fix it.

Thanks to Oleg Grenrus for noticing this.

Merge request reports

Merge request pipeline #11884 passed with warnings

Stage: lint
    Stage: quick-build
      Stage: build
        Stage: full-build
          Stage: cleanup
            Stage: packaging
              Stage: testing

                Merge request pipeline passed with warnings for b23782b4

                Approval is optional

                Closed by Marge BotMarge Bot 5 years ago (Oct 29, 2019 7:48am UTC)

                Merge details

                Activity

                Filter activity
                • Approvals
                • Assignees & reviewers
                • Comments (from bots)
                • Comments (from users)
                • Commits & branches
                • Edits
                • Labels
                • Lock status
                • Mentions
                • Merge request status
                • Tracking
              • Ben Gamari added 2 commits

                added 2 commits

                • 2e70a561 - base: Fix open-file locking
                • d801d3ec - base: Split up file locking implementation

                Compare with previous version

              • Oleg Grenrus resolved all threads

                resolved all threads

              • Ben Gamari added 103 commits

                added 103 commits

                Compare with previous version

              • Ben Gamari added 1 commit

                added 1 commit

                • 4434b780 - base: Split up file locking implementation

                Compare with previous version

              • Ben Gamari added 1 commit

                added 1 commit

                • ebe9487d - base: Split up file locking implementation

                Compare with previous version

              • Ben Gamari added 1 commit

                added 1 commit

                • 243d1abd - base: Split up file locking implementation

                Compare with previous version

              • Ben Gamari added 1 commit

                added 1 commit

                • b23782b4 - base: Split up file locking implementation

                Compare with previous version

              • Ben Gamari assigned to @marge-bot and unassigned @bgamari

                assigned to @marge-bot and unassigned @bgamari

              • Ben Gamari approved this merge request

                approved this merge request

              • Marge Bot
                Marge Bot @marge-bot ·
                Owner

                I will attempt to batch this MR (!2031 (closed))...

              • Marge Bot
                Marge Bot @marge-bot ·
                Owner

                Merged in 78b70e63

              • Marge Bot closed

                closed

                • Please register or sign in to reply