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

Cannot build #700

Closed
PVince81 opened this issue Feb 4, 2019 · 4 comments
Closed

Cannot build #700

PVince81 opened this issue Feb 4, 2019 · 4 comments
Assignees

Comments

@PVince81
Copy link
Contributor

PVince81 commented Feb 4, 2019

Steps

  1. git checkout release-2.4.2
  2. make dist

Expected

Successful build

Actual

Error:

± % make dist
rm -Rf /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist
rm -Rf /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity; mkdir -p /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity
cp -R appinfo css img js l10n lib templates README.md CHANGELOG.md CONTRIBUTING.md /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity
/home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/../../occ integrity:sign-app --privateKey="/home/vincent/.owncloud/certificates/activity.key" --certificate="/home/vincent/.owncloud/certificates/activity.crt" --path="/home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity"
ownCloud is not installed - only a limited number of commands are available
Successfully signed "/home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity"
tar -czf /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity.tar.gz -C /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist activity
tar -cjf /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist/activity.tar.bz2 -C /home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity/build/dist activity
make appstore
make[1]: Entering directory '/home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity'
rm -rf 
mkdir -p 
mkdir: missing operand
Try 'mkdir --help' for more information.
make[1]: *** [Makefile:121: appstore] Error 1
make[1]: Leaving directory '/home/vincent/Private/Work/workspace/owncloud-for-signing/apps/activity'
make: *** [Makefile:115: dist] Error 2

@phil-davis @paurakhsharma please have a look

@PVince81 PVince81 mentioned this issue Feb 4, 2019
25 tasks
@PVince81 PVince81 added this to the development milestone Feb 4, 2019
@PVince81
Copy link
Contributor Author

note: this is blocking the release of the activity app. the release itself isn't urgent but would be good to solve this problem this week

@phil-davis
Copy link
Contributor

phil-davis commented Feb 12, 2019

The problem was introduced in #660 and I have no idea why the PR was approved the way it was, or if the code was accidentally added in a rebase after approval or... - see comment https://github.com/owncloud/activity/pull/660/files#r255856913

See #708 for the "minimal" fix.

@phil-davis
Copy link
Contributor

@PVince81 this should be fixed. Close when you have ttried and got success.

@PVince81
Copy link
Contributor Author

solved, thanks

@patrickjahns patrickjahns removed this from the development milestone Jun 18, 2019
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

3 participants