From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Ted Zlatanov Newsgroups: gmane.emacs.devel Subject: Re: add keyword search and display in details for list-packages Date: Mon, 25 Nov 2013 10:49:07 -0500 Organization: =?utf-8?B?0KLQtdC+0LTQvtGAINCX0LvQsNGC0LDQvdC+0LI=?= @ Cienfuegos Message-ID: <87li0c1nyk.fsf@flea.lifelogs.com> References: <87wql1w6io.fsf@flea.lifelogs.com> <87r4axu4dm.fsf@flea.lifelogs.com> <87k3gorvs4.fsf@flea.lifelogs.com> <87eh6adbqr.fsf@flea.lifelogs.com> <87a9gyc8pp.fsf_-_@flea.lifelogs.com> <87txf5butd.fsf@flea.lifelogs.com> <87y54g8o63.fsf@flea.lifelogs.com> <87fvqk36n5.fsf@flea.lifelogs.com> Reply-To: emacs-devel@gnu.org NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-Trace: ger.gmane.org 1385394513 19087 80.91.229.3 (25 Nov 2013 15:48:33 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 25 Nov 2013 15:48:33 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Mon Nov 25 16:48:38 2013 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1VkyOq-0008Tf-2M for ged-emacs-devel@m.gmane.org; Mon, 25 Nov 2013 16:48:36 +0100 Original-Received: from localhost ([::1]:53396 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkyOp-0002Rj-GM for ged-emacs-devel@m.gmane.org; Mon, 25 Nov 2013 10:48:35 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:54379) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkyOh-0002RL-Jx for emacs-devel@gnu.org; Mon, 25 Nov 2013 10:48:32 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VkyOc-0006LR-GE for emacs-devel@gnu.org; Mon, 25 Nov 2013 10:48:27 -0500 Original-Received: from plane.gmane.org ([80.91.229.3]:60718) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkyOc-0006L1-94 for emacs-devel@gnu.org; Mon, 25 Nov 2013 10:48:22 -0500 Original-Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1VkyOb-0008Ms-2Q for emacs-devel@gnu.org; Mon, 25 Nov 2013 16:48:21 +0100 Original-Received: from c-98-229-61-72.hsd1.ma.comcast.net ([98.229.61.72]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 25 Nov 2013 16:48:21 +0100 Original-Received: from tzz by c-98-229-61-72.hsd1.ma.comcast.net with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 25 Nov 2013 16:48:21 +0100 X-Injected-Via-Gmane: http://gmane.org/ Mail-Followup-To: emacs-devel@gnu.org Original-Lines: 60 Original-X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: c-98-229-61-72.hsd1.ma.comcast.net X-Face: bd.DQ~'29fIs`T_%O%C\g%6jW)yi[zuz6; d4V0`@y-~$#3P_Ng{@m+e4o<4P'#(_GJQ%TT= D}[Ep*b!\e,fBZ'j_+#"Ps?s2!4H2-Y"sx" Mail-Copies-To: never User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.3.50 (gnu/linux) Cancel-Lock: sha1:f0XCbbTEOMvtEOx+fmw9JGUg280= X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 80.91.229.3 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 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.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:165704 Archived-At: --=-=-= Content-Type: text/plain On Mon, 25 Nov 2013 10:20:42 -0500 Stefan Monnier wrote: >> I got `staging/packages/archive-contents' with the right contents for a >> multi-file package (w3) but the single-file package keywords are not >> getting generated. SM> Good start, thanks. >> I spent about 2 hours trying to figure out what's going on, following >> your instructions (which worked) and debugging things ad-hoc. >> It seems that the presence of the "PACKAGE-pkg.el" file is causing the >> keyword evaluation to be skipped as a sort of cache takes over, but >> I lost track of the indirection layers and couldn't figure it out. SM> I can take a look. >> If you could commit my patch and test it once, that would be great. SM> I must have missed your patch. Can you resend it? It was earlier in this thread, re-attaching here. Thanks for looking into it! Ted --=-=-= Content-Type: text/x-diff Content-Disposition: inline; filename=elpa-include-keywords.patch diff --git a/admin/archive-contents.el b/admin/archive-contents.el index 7ee9c1c..8fee66b 100644 --- a/admin/archive-contents.el +++ b/admin/archive-contents.el @@ -187,13 +187,18 @@ Otherwise, return nil." (requires-str (lm-header "package-requires")) (pt (lm-header "package-type")) (simple (if pt (equal pt "simple") (= (length files) 1))) + (keywords (split-string (or (lm-header "keywords") "") + "," t "[ ]+")) (url (or (lm-header "url") (format "http://elpa.gnu.org/packages/%s.html" pkg))) (req (if requires-str (mapcar 'archive--convert-require (car (read-from-string requires-str)))))) - (list simple version description req (list (cons :url url))))))) + (list simple version description req + ;; extra parameters + (list (cons :url url) + (cons :keywords keywords))))))) ((not (file-exists-p pkg-file)) (error "Can find single file nor package desc file in %s" dir))))) --=-=-=--