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

DerClou! recipe - WIP #852

Closed
wants to merge 2 commits into from
Closed

DerClou! recipe - WIP #852

wants to merge 2 commits into from

Conversation

@extrowerk
Copy link
Member

extrowerk commented Nov 1, 2016

Compiles and runs from Terminal but not from Tracker, because it cannot find it files. Fix comes if i find a workaround.

@extrowerk extrowerk force-pushed the extrowerk:derclou branch from 0c678e8 to 440de95 Nov 1, 2016
Copy link
Member

fbrosson left a comment

cosmetic changes

@@ -0,0 +1,79 @@
SUMMARY="A burglary simulation game"
DESCRIPTION="You play Matt Stuvysant, a man of Polish descent and criminal persuasion. \

This comment has been minimized.

Copy link
@fbrosson

fbrosson Nov 1, 2016

Member

The first line of DESCRIPTION exceeds the 80 char recommended limit.
For your convenience, here are the first lines with the "right" line breaks:

DESCRIPTION="You play Matt Stuvysant, a man of Polish descent and criminal \
persuasion. You arrive in London with nothing but the clothes on your back and \
a desire to become the greatest criminal mind this century. Once getting a \
room and meeting some accomplices, you must begin your career. Starting with \
small shops and gradually working your way up to the theft of the Crown Jewels.
app:derclou$secondaryArchSuffix = $portVersion
"

REQUIRES="

This comment has been minimized.

Copy link
@fbrosson

fbrosson Nov 1, 2016

Member

No need for a blank line between PROVIDES and REQUIRES.

devel:libSDL$secondaryArchSuffix
devel:libSDL_mixer$secondaryArchSuffix
"

This comment has been minimized.

Copy link
@fbrosson

fbrosson Nov 1, 2016

Member

No need for a blank line between BUILD_REQUIRES and BUILD_PREREQUIRES.

SOURCE_URI_2="https://sourceforge.net/projects/cosp/files/Original%20Game%20Files/The%20Clue%21%20%2B%20Profidisk%20%28English%29/TheClueProfiData.zip"
CHECKSUM_SHA256="40d1f2df686b9626bee5fdfb691b6937c718b69b9d006001ceed2c947c007f3a"
CHECKSUM_SHA256_2="fe53142dc1c1ae2cd758e0eafa09ad4013027c50a15f88920e8bf2926d5279f3"
SOURCE_DIR="derclou-3d9fc7bfd27503d99c442f8a7564cefb32bb73fd"

This comment has been minimized.

Copy link
@fbrosson

fbrosson Nov 1, 2016

Member

I think you should replace SOURCE_URI by:

srcGitRev="3d9fc7bfd27503d99c442f8a7564cefb32bb73fd"
SOURCE_URI="https://github.com/vcosta/derclou/archive/$srcGitRev.zip"

and change SOURCE_DIR to:

SOURCE_DIR="derclou-$srcGitRev"
@extrowerk extrowerk force-pushed the extrowerk:derclou branch from 440de95 to 5b20d90 Nov 3, 2016
extrowerk added 2 commits Nov 1, 2016
Removed whitespaces

DerClou recipe

Whitespace fix

Fixed recipe, thanks fbrosson
@waddlesplash
Copy link
Member

waddlesplash commented Dec 26, 2016

Fix comes if i find a workaround.

It looks like you added a patch workaround?

@extrowerk
Copy link
Member Author

extrowerk commented Dec 26, 2016

Sadly it doesn't works. I should invert more time int ofixing old PR-s, but this game not really prio-1.

@waddlesplash
Copy link
Member

waddlesplash commented Dec 26, 2016

(added tag on request of @miqlas)

@scottmc scottmc added the wip-no-merge label Mar 3, 2017
@extrowerk
Copy link
Member Author

extrowerk commented May 27, 2017

No interest anymore, closing for now.

@extrowerk extrowerk closed this May 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.