[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/orgit 7d6afa5ae9 60/76: orgit-log-open: Call magit-log-arg
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/orgit 7d6afa5ae9 60/76: orgit-log-open: Call magit-log-arguments from repo |
Date: |
Thu, 13 Jan 2022 13:58:47 -0500 (EST) |
branch: elpa/orgit
commit 7d6afa5ae935ebc8895168529e85268414731214
Author: Kyle Meyer <kyle@kyleam.com>
Commit: Kyle Meyer <kyle@kyleam.com>
orgit-log-open: Call magit-log-arguments from repo
Calling orgit-log-open from an Org buffer that is not in a Git
repository signals a magit-outside-git-repo error that is triggered by
the call to magit-log-arguments. Holding the Magit version constant
at the current master (0f1234e7), this failure bisects to
a2d3937 (Adapt to changes in Magit, 2019-04-17).
Reposition the default-directory binding so that magit-log-arguments
is called within the linked repo.
Note that orgit-rev-open calls magit-diff-arguments, so it _could_ hit
the same failure, but that call already happens with default-directory
correctly bound.
Fixes #34.
---
orgit.el | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/orgit.el b/orgit.el
index f476cfd37e..d843aaf0da 100644
--- a/orgit.el
+++ b/orgit.el
@@ -303,14 +303,14 @@ In that case `orgit-rev-store' stores one or more links
instead."
;;;###autoload
(defun orgit-log-open (path)
(pcase-let* ((`(,repo ,args) (split-string path "::"))
+ (default-directory (orgit--repository-directory repo))
(`(,revs ,args ,files)
(cond ((string-prefix-p "((" args)
(read args))
((string-prefix-p "(" args)
(list (read args) (car (magit-log-arguments))))
(t
- (list (list args) (car (magit-log-arguments))))))
- (default-directory (orgit--repository-directory repo)))
+ (list (list args) (car (magit-log-arguments)))))))
(magit-log-setup-buffer revs args files)))
;;;###autoload
- [nongnu] elpa/orgit e7cddf39e3 56/76: Bump dependencies, (continued)
- [nongnu] elpa/orgit e7cddf39e3 56/76: Bump dependencies, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 3c83adc34a 62/76: orgit--format-export: New function, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 9f09553c09 63/76: orgit-status-store: Account for topics, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 7105c135ce 70/76: Add SPDX-License-Identifier library header, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 26242895ef 72/76: Bump dependencies, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 66367d6bfc 76/76: Release version 1.7.2, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 1b1c8d6963 26/76: use org-link-set-parameters when available, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit f8ca2fb77b 40/76: orgit-abbreviate-hashes: New option, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit f705d581ec 41/76: orgit-rev-store-1: Resolve full hash when not abbreviating, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 0242088fb2 59/76: No longer depend on dash, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 7d6afa5ae9 60/76: orgit-log-open: Call magit-log-arguments from repo,
ELPA Syncer <=
- [nongnu] elpa/orgit a4e689f009 64/76: make: Remove obsolete dependencies, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 609fd0ccfb 68/76: orgit-export: Handle missing local repository explicitly, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit b7eb6de8c6 19/76: orgit-log-store: use first rev as intended, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 78a3932797 45/76: orgit--{current-repository, repository-directory: New functions, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 467f424df4 47/76: orgit-store-repository-id: New option, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit d7e773e1b6 50/76: Add .github/FUNDING.yml, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 9e5f93d516 51/76: Update for Org 9.3, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit 1d257c8cb2 52/76: Change term used for copyright holders, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit a4a565be2a 74/76: Export repository-ID based links, ELPA Syncer, 2022/01/13
- [nongnu] elpa/orgit f956d802f1 75/76: orgit-rev-description-format: New option, ELPA Syncer, 2022/01/13