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

can't compile diorite-4.4.0 on Fedora 25 #16

Closed
martinkg opened this issue Jun 2, 2017 · 1 comment
Closed

can't compile diorite-4.4.0 on Fedora 25 #16

martinkg opened this issue Jun 2, 2017 · 1 comment
Labels
type: bug Something is not working as expected or the documentation is incorrect.
Milestone

Comments

@martinkg
Copy link

martinkg commented Jun 2, 2017

[martin@fc25 SPECS]$ rpmbuild -ba diorite.spec 
Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.zH2FXi
+ umask 022
+ cd /home/martin/rpmbuild/BUILD
+ cd /home/martin/rpmbuild/BUILD
+ rm -rf diorite-4.4.0
+ /usr/bin/gzip -dc /home/martin/rpmbuild/SOURCES/diorite-4.4.0.tar.gz
+ /usr/bin/tar -xof -
+ STATUS=0
+ '[' 0 -ne 0 ']'
+ cd diorite-4.4.0
+ /usr/bin/chmod -Rf a+rX,u+w,g-w,o-w .
+ rm ./waf
+ rm -rf ./.waf
+ sed -i -e 's|#!/usr/bin/env python3|#!/usr/bin/python3|g' testgen.py
+ exit 0
Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.JR5em4
+ umask 022
+ cd /home/martin/rpmbuild/BUILD
+ cd diorite-4.4.0
+ waf-3 configure --prefix=/usr --libdir=/usr/lib64 --novaladoc
Setting top to                           : /home/martin/rpmbuild/BUILD/diorite-4.4.0 
Setting out to                           : /home/martin/rpmbuild/BUILD/diorite-4.4.0/build 
fatal: Not a git repository (or any of the parent directories): .git
Traceback (most recent call last):
  File "/usr/share/waf3/waflib/Scripting.py", line 121, in waf_entry_point
    run_commands()
  File "/usr/share/waf3/waflib/Scripting.py", line 182, in run_commands
    ctx=run_command(cmd_name)
  File "/usr/share/waf3/waflib/Scripting.py", line 173, in run_command
    ctx.execute()
  File "/usr/share/waf3/waflib/Configure.py", line 84, in execute
    super(ConfigurationContext,self).execute()
  File "/usr/share/waf3/waflib/Context.py", line 87, in execute
    self.recurse([os.path.dirname(g_module.root_path)])
  File "/usr/share/waf3/waflib/Context.py", line 128, in recurse
    user_function(self)
  File "/home/martin/rpmbuild/BUILD/diorite-4.4.0/wscript", line 146, in configure
    add_version_info(ctx)
  File "/home/martin/rpmbuild/BUILD/diorite-4.4.0/wscript", line 61, in add_version_info
    bare_version, n_commits, revision_id = get_git_version()
ValueError: not enough values to unpack (expected 3, got 1)
error: Bad exit status from /var/tmp/rpm-tmp.JR5em4 (%build)

only works if i use this patch:

[root@fc25 tmp]# cat -v /home/martin/rpmbuild/SOURCES/wscript.patch
--- diorite-4.4.0.orig/wscript	2017-05-27 16:36:46.000000000 +0200
+++ diorite-4.4.0/wscript	2017-05-31 09:05:17.753873846 +0200
@@ -50,11 +50,6 @@
 
 
 def get_git_version():
-	import subprocess
-	try:
-		output = subprocess.Popen(["git", "describe", "--tags", "--long"], stdout=subprocess.PIPE).communicate()[0]
-		return output.decode("utf-8").strip().split("-")
-	except Exception as e:
 		return VERSION, "0", REVISION_SNAPSHOT
 
 def add_version_info(ctx):

waf-python3-1.9.11-1.fc25 package with /usr/bin/waf-3 is installed.

jiri-janousek added a commit to tiliado/nuvolaplayer that referenced this issue Jun 2, 2017
jiri-janousek added a commit that referenced this issue Jun 2, 2017
Issue: #16

Signed-off-by: Jiří Janoušek <[email protected]>
@jiri-janousek
Copy link
Member

Thanks for reporting the issue. It also occurs in Nuvola. I've fixed both in master branches.

@jiri-janousek jiri-janousek added this to the 4.5 milestone Jun 2, 2017
@jiri-janousek jiri-janousek added the type: bug Something is not working as expected or the documentation is incorrect. label Jun 2, 2017
jiri-janousek added a commit to tiliado/nuvolaplayer that referenced this issue Jun 25, 2017
New Features:

  * Nuvola Apps Runtime supports the integration of a progress bar and
    volume management. Web app scripts which use this feature can not
    only provide track length & position and current volume but also
    allow user to change that remotely, e.g. from Media Player GNOME
    Shell extension. At the present, only Deezer and Google Play Music
    scripts use these features but others will follow.
    Issue: #22 Issue: #155
  * If Nuvola Apps Runtime detect a Nvidia graphics card, it checks
    whether the flatpak extension with corresponding graphics driver is
    installed. If it isn't, e.g. because of a bug in GNOME Software,
    an error message is shown to provide the user with installation
    instructions. Issue: #342
  * After a lot of effort, a workaround for the instability of Flash
    plugin was found out and is used until WebKitGTK developers find a
    proper fix. However, it is applied only in flatpak builds because it
    may have negative impact on other WebKitGTK applications otherwise.
    Issue: #354

Bug fixes:

  * Wrong command in desktop launcher was fixed.
    Issue: #348
  * Fix wscript for non-git builds. Issue: tiliado/diorite#16

News for Script Maintainers:

  * `Nuvola.VERSION_MICRO` contains micro version of Nuvola Runtime.
  * `Nuvola.API_VERSION_MAJOR` and `Nuvola.API_VERSION_MINOR` are now
    deprecated aliases of `Nuvola.VERSION_MAJOR`
    and `Nuvola.VERSION_MINOR`.
  * Tutorial was updated to use Nuvola ADK 4.4.
  * Added documentation of web app requirement flags.
  * Added documentation of user agent quirks.
  * New API for progress bar integration.
  * New API for volume management integration.
  * New utility functions `Nuvola.encodeVersion` and
    `Nuvola.checkVersion`.
  * `Nuvola.triggerMouseEvent` and `clickOnElement` support relative
    x & y coordinates.

Under the Hood:

  * New dependencies: libdrm >= 2.2 and libdri2 >= 1.0
  * Nuvola checks whether VDPAU and VA-API drivers are installed and
    prints debugging information to console. It will show error dialog
    in the future though, so make sure the drivers are installed.
    Issue: #280
  * Internal icon loading code was refactored. Legacy icon.png and
    nuvolaplayer3_XXX icons are no longer supported.
    eu.tiliado.NuvolaAppXxx is used everywhere.
    Issue: #353

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit that referenced this issue Jun 25, 2017
  * Fix wscript for non-git builds. Issue: #16
  * Dioritedb has been refactored significantly.
  * Bundled glib.vapi is no longer used.
  * Various utility functions were added, see git log for details.

Signed-off-by: Jiří Janoušek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something is not working as expected or the documentation is incorrect.
Projects
None yet
Development

No branches or pull requests

2 participants