From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.ciao.gmane.io!not-for-mail From: Adam Porter Newsgroups: gmane.emacs.devel Subject: Re: On elisp running native Date: Thu, 12 Mar 2020 12:10:16 -0500 Message-ID: References: <83tv5mp48l.fsf@gnu.org> <83sgl0lchm.fsf@gnu.org> <83imlwl9vm.fsf@gnu.org> <83o8uegykm.fsf@gnu.org> <74dd94a9-28cb-a5fd-dbc7-ab21009834ad@cs.ucla.edu> <87mu8vjcmm.fsf@alphapapa.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Injection-Info: ciao.gmane.io; posting-host="ciao.gmane.io:159.69.161.202"; logging-data="118420"; mail-complaints-to="usenet@ciao.gmane.io" To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Thu Mar 12 18:17:19 2020 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jCRSR-000UhY-H1 for ged-emacs-devel@m.gmane-mx.org; Thu, 12 Mar 2020 18:17:19 +0100 Original-Received: from localhost ([::1]:46380 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jCRSQ-0001um-5h for ged-emacs-devel@m.gmane-mx.org; Thu, 12 Mar 2020 13:17:18 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:40585) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jCRLr-0001HG-MK for emacs-devel@gnu.org; Thu, 12 Mar 2020 13:10:32 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jCRLq-0003XC-IR for emacs-devel@gnu.org; Thu, 12 Mar 2020 13:10:31 -0400 Original-Received: from mail-ed1-f50.google.com ([209.85.208.50]:35477) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1jCRLq-0003W1-CF for emacs-devel@gnu.org; Thu, 12 Mar 2020 13:10:30 -0400 Original-Received: by mail-ed1-f50.google.com with SMTP id a20so8410897edj.2 for ; Thu, 12 Mar 2020 10:10:29 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=BOZebdgw5h/e1HnRhyBN2Usj854WCUwrAMijeFzJI48=; b=URiZprdMY/XsE0WkauERcyTSiAjS3yKsckHKBtEUM83U5Ctxhe+dbsxejxLLw/x31o 76HgH5fuLBvZWzSmnsyC7kCrBmu2kuRSO1RpdShzwQ64BFp3ENpjPlrhl8yeCxxuIrWE C7eNlgAZErU/FxjxBKzg67v3ItwfrFC3pH3FWIouSFblReva5s1HS6/hm5korQYOg0M6 aPBAlgVFlOdsTnJwL57zIy2+VJnYZOOt07k+WX0iiL8pUvipYUYaUyheCoycDvTk9grQ 1YTr5tENnFzvpFL1IuDVyBGY+HYdDRq8orgizwl+/v2GH8XDvsk4Rf/b5T+MTEtTmmpL k9sw== X-Gm-Message-State: ANhLgQ2hc5yA0fhotYWHf4/0SC/HrlTUMfNW1Z51W5EjfsS7n192wxdg 7ru+INe7U5Jo94MEsHG7Ispv+M1wSuL8qJ5caWWCUQ== X-Google-Smtp-Source: ADFU+vv8hgCvn8VVRLt+TR7lzUdjCEvur+pnu0zCio0ZI7CgRSffDV+KMtRrAZ7l0MN7U9TWCTjOC1TSdfndkR9lEwo= X-Received: by 2002:aa7:de13:: with SMTP id h19mr9275378edv.52.1584033027535; Thu, 12 Mar 2020 10:10:27 -0700 (PDT) In-Reply-To: X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 209.85.208.50 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:245490 Archived-At: Hi Andrea, On Wed, Mar 11, 2020 at 5:04 PM Andrea Corallo wrote: > The reason why your code is byte-compiled correctly during the package > installation is that apparently package.el is loading the non compiled > version *before* running the byte-compiler through > `package--load-files-for-activation'. As a consequence when the > byte-compiler runs the defvar already took effect. > > I think your fix to wrap the defvar is correct. > > That said we have to fix the native compiler that not only is not > signaling in a sufficiently visible way the error, but is also producing > an eln just ignoring it :) > > I'll look into this second point as next. Thanks very much for digging into that. Glad to know that I was indeed doing something at least somewhat wrong, and how to fix it.