Skip to content

packaging update#279

Merged
jacalata merged 8 commits into
mainfrom
jac/windows-spec-update
May 9, 2024
Merged

packaging update#279
jacalata merged 8 commits into
mainfrom
jac/windows-spec-update

Conversation

@jacalata

Copy link
Copy Markdown
Contributor

Building with pyinstaller v6 caused the executable to trigger automated malware alerts. Changing back to build with v5.13 gets an almost clean scan.

tabcmd 0.13 from github built with pyinstaller 6.x: ~30 vendors flagged it
https://www.virustotal.com/gui/file/829b092e47228c0cea2c54e0f75e63501b5e12135a76fef2dcbaba2deac228ed

my freshly built copy of 0.13 with pyinstaller 5.13: 2 vendors flagged it
https://www.virustotal.com/gui/file/b1c8be24ade44a8eae93e6a03d56fcd99caf389549c647fcb133b965d1c8e1f4/detection

tabcmd0.11.exe from github: was also built with 5.13: 1 vendor flaggged it
https://www.virustotal.com/gui/file/f6453b2549b9f696701e76d83b7c349a8710d3492a4fabf3029a6634c7f236f2

Comment thread contributing.md Outdated
####To work with tabcmd, you need to have **Python 3.7+** installed.
## Install Tabcmd
> [!NOTE]
> These instructions are for people who want to write code in or using tabcmd. If > you are interested in tabcmd but not the code, see [here](Readme.md).

@anyoung-tableau anyoung-tableau May 6, 2024

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm having some trouble understanding this line.

Also, I think there's a stray > in the 2nd sentence.

Comment thread contributing.md Outdated
jacalata and others added 2 commits May 8, 2024 14:36
Co-authored-by: Andy Young <90729701+anyoung-tableau@users.noreply.github.com>
@jacalata
jacalata merged commit 19aabf5 into main May 9, 2024
@jacalata
jacalata deleted the jac/windows-spec-update branch January 3, 2025 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants