Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requesting a Mac friendly build #1

Open
vivekr-gh opened this issue Jun 18, 2015 · 1 comment
Open

Requesting a Mac friendly build #1

vivekr-gh opened this issue Jun 18, 2015 · 1 comment

Comments

@vivekr-gh
Copy link

Current build copies blast executables to distribution and these end with the ".exe" extension. On a mac, this prevents the editor from running blast since it only looks for ".exe" for windows operating systems:

com.quattroresearch.blastws.LocalConfig:

  private static final String BLASTN = findFile("blastn");
  private static final String BLASTP = findFile("blastp");
  [...]
    if (OSUtils.IS_WINDOWS) {
      exeFileName = fileName + ".exe";

One simple way to fix this might simply be to add ".exe" to your string and changing the conditional to look for anything that isn't windows.

  private static final String BLASTN = findFile("blastn.exe");
  private static final String BLASTP = findFile("blastp.exe");
  [...]
    if (OSUtils.IS_WINDOWS==false) {
      exeFileName = fileName.substring(0,fileName.length-4);

That's one way. Another way might be to detect the OS in ant and remove the file extension if not windows. Multiple ways to achieve this result :)

@MarkusWeisser
Copy link
Contributor

Yes, you are right. You have to manually copy the blast executables on Mac. We will check your proposed solution.

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

No branches or pull requests

2 participants