Re: [patch] Refactor: clipboard, translations, jasmine

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Atira Odhner <aodhner(at)pivotal(dot)io>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [patch] Refactor: clipboard, translations, jasmine
Date: 2017-03-10 15:26:58
Message-ID: CA+OCxox8_0sX2pZVSD+OjCeR_Qva=6sh=CPT5NkzLUQ7gc0xTA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Fri, Mar 10, 2017 at 2:53 PM, Atira Odhner <aodhner(at)pivotal(dot)io> wrote:
>> Actually that's not an issue - the package builds will (in most cases
>> now) remove the test directories, and will only include files that are
>> in GIT anyway. Please move everything under web/regression as right
>> now it's cluttering up the root directory which doesn't make sense as
>> it only applies to the web/ code.
>
>
> I just did a build and saw python tests showing up in the mac-build folder,
> before the dmg is created. Can you point out where the exclusion of test
> directories is supposed to happen during the dmg build process so I can
> check that it works in this case?

Well I did say "in most cases now" :-)

For Windows we have:

FOR /R "%PGBUILDPATH%\web" %%f in (tests feature_tests __pycache__) do
RD /Q /S "%%f"

And I thought I'd also done the python wheel as well, but apparently
not (though now I think about it, that was intentional). In any case,
I'm not too worried about that, as a) we will exclude them from
packages where appropriate even if we don't yet, and b) it's only a
few files.

The important thing is that the node_modules directory isn't included
(and thinking about it, the snapshots directory and regression log).
That's over 7K files :-o. The wheel and source packages will exclude
that, but we may need to explicitly exclude it from the Windows and
macOS builds.

I'll put that on my TODO.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2017-03-10 15:49:57 pgAdmin 4 commit: Resolved an issue finding the python interpreter on *
Previous Message Juliano 2017-03-10 15:12:21 "Cancel Query" or "Terminate backend" - Pgadmin4