From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mx.groups.io with SMTP id smtpd.web09.1945.1572496765221678069 for ; Wed, 30 Oct 2019 21:39:25 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.31, mailfrom: michael.d.kinney@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Oct 2019 21:39:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,250,1569308400"; d="scan'208";a="375115547" Received: from orsmsx108.amr.corp.intel.com ([10.22.240.6]) by orsmga005.jf.intel.com with ESMTP; 30 Oct 2019 21:39:24 -0700 Received: from orsmsx126.amr.corp.intel.com (10.22.240.126) by ORSMSX108.amr.corp.intel.com (10.22.240.6) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 30 Oct 2019 21:39:24 -0700 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.28]) by ORSMSX126.amr.corp.intel.com ([169.254.4.48]) with mapi id 14.03.0439.000; Wed, 30 Oct 2019 21:39:24 -0700 From: "Michael D Kinney" To: "devel@edk2.groups.io" , "leif.lindholm@linaro.org" , "sean.brogan@microsoft.com" , "Kinney, Michael D" Subject: Re: [edk2-devel] [Patch v3 03/22] requirements.txt: Add python pip requirements file Thread-Topic: [edk2-devel] [Patch v3 03/22] requirements.txt: Add python pip requirements file Thread-Index: AQHVjsmqUGcYP6gUyEWI/TB7lBseQady8f8AgAAC2wCAATdokA== Date: Thu, 31 Oct 2019 04:39:23 +0000 Message-ID: References: <20191030022757.GS16820@bivouac.eciton.net> <2748.1572403968433015098@groups.io> <20191030030302.GW16820@bivouac.eciton.net> In-Reply-To: <20191030030302.GW16820@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.22.254.138] MIME-Version: 1.0 Return-Path: michael.d.kinney@intel.com Content-Language: en-US Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Hi Leif, I will see if we can add a proper file header with a description. I may break this out into its own patch since it is useful to install these pip modules to run CI tests and builds locally. The pip modules are not only use in a CI agent. Mike > -----Original Message----- > From: devel@edk2.groups.io On > Behalf Of Leif Lindholm > Sent: Tuesday, October 29, 2019 8:03 PM > To: devel@edk2.groups.io; sean.brogan@microsoft.com > Subject: Re: [edk2-devel] [Patch v3 03/22] > requirements.txt: Add python pip requirements file >=20 > OK, if it makes a difference for tools (and security > updates), let's try to keep it. (*grumble*) >=20 > *But* given its too-generic name, can we add a big bold > comment header to the file explaining what it is? > Some quick searching suggests lines starting with # are > ignored, so hopefully this should be possible? >=20 > Best Regards, >=20 > Leif >=20 > On Tue, Oct 29, 2019 at 07:52:48PM -0700, Sean via > Groups.Io wrote: > > It is a convention for projects using python.=A0 It > definitely isn't required but there are some features > that come for free when using that filename. > > > > https://github.blog/2018-07-12-security-vulnerability- > alerts-for-pytho > > n/ > > and > > https://help.github.com/en/github/visualizing- > repository-data-with-gra > > phs/listing-the-packages-that-a-repository-depends-on > > > > You can also get more insight from github into > dependencies and dependents. > > example: > > https://github.com/tianocore/edk2-pytool- > extensions/network/dependenci > > es > > > > I did notice on the example from above that github > picked up the requirements.publisher.txt so i don't know > what their pattern matching does exactly. > > > > > > >=20 >=20