Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lHmp9-0007mm-2m for pgadmin-hackers@arkaria.postgresql.org; Thu, 04 Mar 2021 12:11:23 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1lHmp4-0005kp-OH for pgadmin-hackers@arkaria.postgresql.org; Thu, 04 Mar 2021 12:11:18 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lHkwz-00066Q-Qz for pgadmin-hackers@lists.postgresql.org; Thu, 04 Mar 2021 10:11:21 +0000 Received: from mail-wm1-x331.google.com ([2a00:1450:4864:20::331]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1lHkwx-0002cd-HV for pgadmin-hackers@postgresql.org; Thu, 04 Mar 2021 10:11:20 +0000 Received: by mail-wm1-x331.google.com with SMTP id u125so9035103wmg.4 for ; Thu, 04 Mar 2021 02:11:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:in-reply-to:references:mime-version :content-transfer-encoding; bh=3JfcjISfO+pNtlOJn8VkysJALTVqSeYc1kvc9+sORIg=; b=lnRUOZjMBXVWfpA/NzF1JrbdEasZjQA1ZJrRsNBI4toeYNncn78jtpd3/Y7rhrqlex U9U+KbWIikRalqaOvCmnbdQ3Fa+GmnhRgbIA7BzFOcXk/BmDxjaOcE2oEy2yDzhCMFe8 NaWxe9zPT7pawDV4J3IELHIT81cSAFVqJ0lg2i4CXh4mPDoKmHc3zfe4hC+r73wW1LhC EmWp7LYEbv9P5dk/1c3B9NoP8XpeI0vj5hcCujcq5mybK5YVlGvTeVqpF9Oq+LW35k3S qizUZLntsDNg+msexsN9p6BMkrAuJYygFO5Q1nh7gaZkC3tJ0oNFLaKw9bMq1MW5pfbI 740A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=3JfcjISfO+pNtlOJn8VkysJALTVqSeYc1kvc9+sORIg=; b=o1xxfD3vDYXN9SCvFkuVNKCky5kYwXZM0KEAokWUbEpS4kcf3RT0CqePTcnlaZ5Kib Y6gYrELnE9ZEwAhdJeb3mrmfbMKotSvEkD2R+VC8Qs/KuwUFGUaiyMhSvBt5YAUgXN/+ HNNUMSKSV9MUPonbwQ1eomJrFsh7E5CKltKTvwQFRb/KY7mglNQT00IWQeXUDlsqfghh AaPtEK+MXOAO06/pdExBaR70Y+bUnsxm6CkQ5/gIHw5qk0aqWRgjAWGkMgs9k+TicLq5 /1dX+bxaOj2UhtHyf3I4xYEAcR285KH2jDvRHLvHYjsClr5h3liZnX8yKUEHpB7+36/Z 4cJQ== X-Gm-Message-State: AOAM532GqPbGWcELHebKNwg+YvKfRn/RAdEF87Z0Oe0WUfyjDXdlgDT4 D7fQH7skWhCXAoiAdZEE+EVKJzhRE/+1wA== X-Google-Smtp-Source: ABdhPJzICF8bUW/3sEYTdL0ZPD6/e0kt9o89odnim1FFSDdD27PMFevcAKypdjQWcDxAU1aPzrTMNQ== X-Received: by 2002:a7b:cb05:: with SMTP id u5mr3143631wmj.46.1614852677156; Thu, 04 Mar 2021 02:11:17 -0800 (PST) Received: from baba.localnet (79-73-191-104.dynamic.dsl.as9105.com. [79.73.191.104]) by smtp.gmail.com with ESMTPSA id j125sm8952292wmb.44.2021.03.04.02.11.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Mar 2021 02:11:16 -0800 (PST) From: Shaheed Haque To: pgadmin-hackers , Dave Page Subject: Re: Making Kerberos optional in the Python wheel Date: Thu, 04 Mar 2021 10:06:57 +0000 Message-ID: <2770352.mvXUDI8C0e@baba> In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk A big thumbs up from me at least :-). On Thursday, 4 March 2021 10:02:00 GMT Dave Page wrote: > There have been a couple of complaints that the latest Python wheel > distribution doesn't install cleanly. This happens when there is no > pre-built gssapi wheel on PyPi that matches the users combination of Python > version and platform, *and* the MIT Kerberos development headers etc. are > not present on the system, so the source wheel cannot be compiled. > > This seems like it's a bit onerous on users, especially if they're on > Windows where they'll also need a suitable compiler to be installed. The > attached patch aims to address that by making the Kerberos support optional > (thankfully, Khushboo made the code handle lack of gssapi libraries). > > To install without gssapi, users would simply do: > > pip install pgadmin4 > > or > > pip install pip install /path/to/pgadmin4-5.0-py3-none-any.whl > > To install with gssapi: > > pip install pgadmin4['kerberos'] > > or > > pip install pip install /path/to/pgadmin4-5.0-py3-none-any.whl['kerberos'] > > The patch also cleans up some old cruft that was required for now > unsupported Python versions. > > Thoughts?