Invalid face box

[joostkremers]

I just opened magit-status and was greeted with the following error:

"Invalid face box" :line-width (-1 . -1) :style released-button

Running magit-status a second time works; the error only seems to appear the first time I call magit-status in an Emacs session.

The reason I'm posting the issue here is that it seems to be related to the forge package. Installing Magit and Forge in an Emacs sandbox (https://github.com/alphapapa/emacs-sandbox.sh) gives the same error during installation of Forge:

Compiling file /tmp/tmp.cCvhHLcFmJ/elpa/forge-20210506.1802/forge-bitbucket.el at Fri May  7 17:38:46 2021
Entering directory ‘/tmp/tmp.cCvhHLcFmJ/elpa/forge-20210506.1802/’
/usr/bin/cc -I/tmp/tmp.cCvhHLcFmJ/elpa/emacsql-sqlite-20190727.1710/sqlite -O2 -DSQLITE_THREADSAFE=0 -DSQLITE_DEFAULT_FOREIGN_KEYS=1 -DSQLITE_ENABLE_FTS5 -DSQLITE_ENABLE_FTS4 -DSQLITE_ENABLE_FTS3_PARENTHESIS -DSQLITE_ENABLE_RTREE -DSQLITE_ENABLE_JSON1 -DSQLITE_SOUNDEX /tmp/tmp.cCvhHLcFmJ/elpa/emacsql-sqlite-20190727.1710/sqlite/sqlite3.c /tmp/tmp.cCvhHLcFmJ/elpa/emacsql-sqlite-20190727.1710/sqlite/emacsql.c -lm -ldl -o /tmp/tmp.cCvhHLcFmJ/elpa/emacsql-sqlite-20190727.1710/sqlite/emacsql-sqlite
forge-bitbucket.el:27:1:Error: Invalid face box: :line-width, (-1 . -1), :style, released-button

Installing only Magit in the sandboxed Emacs doesn't produce any error.

In my regular Emacs (not the sandbox), calling magit-status the first time in a session yields the following backtrace:

Debugger entered--Lisp error: (error "Invalid face box" :line-width (-1 . -1) :style released-button)
  internal-set-lisp-face-attribute(forge-topic-label :box (:line-width (-1 . -1) :style released-button) #<frame emacs@Swift 0x562d4e2e9ec0>)
  set-face-attribute(forge-topic-label #<frame emacs@Swift 0x562d4e2e9ec0> :box (:line-width (-1 . -1) :style released-button))
  apply(set-face-attribute forge-topic-label #<frame emacs@Swift 0x562d4e2e9ec0> (:box (:line-width (-1 . -1) :style released-button)))
  face-spec-set-2(forge-topic-label #<frame emacs@Swift 0x562d4e2e9ec0> (:box (:line-width (-1 . -1) :style released-button)))
  face-spec-recalc(forge-topic-label #<frame emacs@Swift 0x562d4e2e9ec0>)
  face-spec-set(forge-topic-label ((t :box (:line-width (-1 . -1) :style released-button))) face-defface-spec)
  custom-declare-face(forge-topic-label ((t :box (:line-width (-1 . -1) :style released-button))) "Face used for topic labels." :group forge-faces)
  byte-code("\300\301!\210\302\301\303\304#\210\305\306\307\310\311\312%\210\305\313\314\315\311\312%\210\305\316\317\320\311\312%\210\305\321\322\323\311\312%\210\305\324\325\326\311\312%\210..." [make-variable-buffer-local forge-display-in-status-buffer put permanent-local t custom-declare-face forge-topic-unread ((t :inherit bold)) "Face used for title of unread topics." :group forge-faces forge-topic-closed ((t :inherit magit-dimmed)) "Face used for title of closed topics." forge-topic-open ((t :inherit default)) "Face used for title of open topics." forge-topic-merged ((t :inherit magit-dimmed)) "Face used for number of merged pull-requests." forge-topic-unmerged ((t :inherit magit-dimmed :slant italic)) "Face used for number of unmerged pull-requests." forge-topic-label ((t :box (:line-width (-1 . -1) :style released-button))) "Face used for topic labels." forge-post-author ((t :inherit bold)) "Face used for post author in topic view." forge-post-date ((t :inherit italic)) "Face used for post date in topic view." defalias forge-topic-p eieio-make-class-predicate forge-topic forge-topic--eieio-childp eieio-make-child-predicate forge-topic-child-p make-obsolete "use (cl-typep ... \\='forge-topic) instead" "25.1" define-symbol-prop cl-deftype-satisfies eieio-defclass-internal (forge-post) nil (:abstract t)] 6)
  require(forge-topic)
  byte-code("\304\305!\210\304\306!\210\304\307!\210\310\311!\210\304\312!\210\304\313!\210\304\314!\210\304\315!\210\304\316!\210\304\317!\210\304\320!\210\304\321!\210\304\322..." [forge--sqlite-available-p magit-commit-section-map magit-remote-section-map magit-branch-section-map require magit forge-db forge-core provide forge forge-repo forge-post forge-topic forge-issue forge-pullreq forge-revnote forge-notify forge-github forge-gitlab forge-gitea forge-gogs forge-bitbucket forge-semi forge-commands forge-list magit-add-section-hook magit-status-sections-hook forge-insert-pullreqs nil t forge-insert-issues eval-after-load magit-mode #f(compiled-function () #<bytecode 0x158b53effd39>) define-key [remap magit-browse-thing] forge-browse-dwim [remap magit-browse-thing] forge-browse-remote [remap magit-browse-thing] forge-browse-branch "\3\26" forge-visit-topic transient-append-suffix magit-dispatch "%" ("'" "Forge" forge-dispatch) magit-fetch "m" ("y" "forge topics" forge-pull) ...] 5)
  require(forge nil t)
  (not (require 'forge nil t))
  (if (not (require 'forge nil t)) (display-warning 'use-package (format "Cannot load %s" 'forge) :error) (condition-case err (progn (setq forge-pull-notifications t) t) ((debug error) (funcall use-package--warning69 :config err))))
  (lambda nil (if (not (require 'forge nil t)) (display-warning 'use-package (format "Cannot load %s" 'forge) :error) (condition-case err (progn (setq forge-pull-notifications t) t) ((debug error) (funcall use-package--warning69 :config err)))))()
  eval-after-load-helper("/home/joost/.emacs.d/elpa/magit-20210430.404/magit...")
  run-hook-with-args(eval-after-load-helper "/home/joost/.emacs.d/elpa/magit-20210430.404/magit...")
  do-after-load-evaluation("/home/joost/.emacs.d/elpa/magit-20210430.404/magit...")
  require(magit)
  byte-code("\301\302!\210\303\304\305\306\307\310\311\312&\7\210\313\314\315\316\317DD\320\311\304\321\322&\7\210\313\323\315\316\324DD\325\326\327\311\304\321\322\330\331&\13\210\313..." [magit-log-margin--custom-type require magit custom-declare-group magit-status nil "Inspect and manipulate Git repositories." :link (info-link "(magit)Status Buffer") :group magit-modes custom-declare-variable magit-status-mode-hook funcall function #f(compiled-function () #<bytecode 0x158b53e66c75>) "Hook run after entering Magit-Status mode." :type hook magit-status-headers-hook #f(compiled-function () #<bytecode 0x158b53e66c85>) "Hook run to insert headers into the status buffer...." :package-version (magit . "2.1.0") :options (magit-insert-error-header magit-insert-diff-filter-header magit-insert-repo-header magit-insert-remote-header magit-insert-head-branch-header magit-insert-upstream-branch-header magit-insert-push-branch-header magit-insert-tags-header) magit-status-sections-hook #f(compiled-function () #<bytecode 0x158b53e66c95>) "Hook run to insert sections into a status buffer." (magit . "2.12.0") magit-status-initial-section #f(compiled-function () #<bytecode 0x158b53e66ca5>) "The section point is placed on when a status buffe..." (magit . "2.90.0") (choice (const :tag "as usual" nil) (repeat (choice (number :tag "nth top-level section") (sexp :tag "section identity")))) magit-status-goto-file-position #f(compiled-function () #<bytecode 0x158b53e66cb5>) "Whether to go to position corresponding to file po..." (magit . "3.0.0") boolean magit-status-show-hashes-in-headers #f(compiled-function () #<bytecode 0x158b53e66cc5>) "Whether headers in the status buffer show hashes.\n..." (magit . "2.4.0") magit-status-margin #f(compiled-function () #<bytecode 0x158b53e66cdd>) "Format of the margin in `magit-status-mode' buffer..." (magit . "2.9.0") magit-margin :initialize ...] 20)
  autoload-do-load((autoload "magit-status" "Show the status of the current Git repository in a..." t nil) magit-status)
  command-execute(magit-status record)
  execute-extended-command(nil "magit-status" "magi")
  funcall-interactively(execute-extended-command nil "magit-status" "magi")
  call-interactively(execute-extended-command nil nil)
  command-execute(execute-extended-command)