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

build.properties: remove non-existing src folder #65

Closed
wants to merge 1 commit into from

Conversation

jukzi
Copy link

@jukzi jukzi commented Nov 29, 2023

No description provided.

@akurtakov
Copy link
Member

This patch seems to have totally broken the build - aka src folder is used?

@jukzi
Copy link
Author

jukzi commented Nov 29, 2023

indeed

15:30:59  [INFO] Executing tasks
15:30:59  [INFO]      [copy] Copying 693 files to /home/jenkins/agent/workspace/ipse.platform.swt.binaries_PR-65/eclipse.platform.swt.binaries/bundles/org.eclipse.swt.win32.win32.x86_64/src

@jukzi jukzi closed this Nov 29, 2023
@HeikoKlare
Copy link
Contributor

HeikoKlare commented Nov 29, 2023

The ANT tasks of the build copy the OS-dependent source files to the according OS fragment's src folder:

15:30:59 [INFO] Executing tasks
15:30:59 [INFO] [copy] Copying 693 files to /home/jenkins/agent/workspace/ipse.platform.swt.binaries_PR-65/eclipse.platform.swt.binaries/bundles/org.eclipse.swt.win32.win32.x86_64/src

When removing the folder from the build.properties, probably the resulting bundle will be empty and thus the classes cannot be found.

@jukzi jukzi deleted the output branch November 29, 2023 15:54
@laeubi
Copy link
Contributor

laeubi commented Nov 29, 2023

Please dont mess with the buildfiles for SWT as these are build in a special way with ant not maven...

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.

5 participants