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

[boot] mthemedaemon service files with new boot targets #2

Merged
merged 6 commits into from Jul 8, 2013
Merged

Conversation

xavinux
Copy link
Contributor

@xavinux xavinux commented Jul 5, 2013

No description provided.

YAMLFILE = $(addsuffix .yaml, $(PKG_NAME))


include /usr/share/packaging-tools/Makefile.common
Copy link

Choose a reason for hiding this comment

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

What is this makefile for? We don't generally have them

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Warning: There is no "Makefile" for this package, please update it using packaging-tools
Q: Need to create Makefile from default template?(y/N) y

When this question appears after running the specify command, I answer yes as didn't know that is not necessary.

@special
Copy link

special commented Jul 6, 2013

@plundstr should look over this on Monday too. Thanks.

@xavinux
Copy link
Contributor Author

xavinux commented Jul 6, 2013

I will wait until Monday to @plundstr look over it.

Delete mthemedaemon Makefile as is not necessary.
@plundstr
Copy link
Contributor

plundstr commented Jul 8, 2013

Why you have removed /var/cache/meegotouch???

@xavinux
Copy link
Contributor Author

xavinux commented Jul 8, 2013

Add xorg.service to the .service file and /var/cache/meegotouch to the .spec file

@plundstr
Copy link
Contributor

plundstr commented Jul 8, 2013

take out xorg.service. It should not be mentioned here (pre-user-session.target incldues it)
You have two if stamenets in postun section btu only one "fi"

@plundstr
Copy link
Contributor

plundstr commented Jul 8, 2013

There is still problem with if stamenets in postun (ifs should not be nested). I will fix it myself after merge

plundstr pushed a commit that referenced this pull request Jul 8, 2013
[boot] mthemedaemon service files with new boot targets
@plundstr plundstr merged commit 78e292f into nemomobile-graveyard:master Jul 8, 2013
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.

None yet

3 participants