Bug 103496 - svga_screen.c:26:46: error: git_sha1.h: No such file or directory
Summary: svga_screen.c:26:46: error: git_sha1.h: No such file or directory
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: Mesa core (show other bugs)
Version: git
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: mesa-dev
QA Contact: mesa-dev
URL:
Whiteboard:
Keywords: bisected, regression
Depends on:
Blocks:
 
Reported: 2017-10-28 09:25 UTC by Vinson Lee
Modified: 2017-10-30 22:25 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:

Attachments

Description Vinson Lee 2017-10-28 09:25:52 UTC 
mesa: 05592cebd4fd (master 17.4.0-devel)

SCons build regression

  Compiling src/gallium/drivers/svga/svga_screen.c ...
src/gallium/drivers/svga/svga_screen.c:26:46: error: git_sha1.h: No such file or directory

commit 7088622e5fb506b64c906b673a9af0feb9a5ba14
Author: Eric Engestrom <eric.engestrom@imgtec.com>
Date:   Wed Oct 25 14:04:35 2017 +0100

    buildsys: move file regeneration logic to the script itself
    
    Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com>
    Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
    Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Comment 1 Brian Paul 2017-10-30 17:01:37 UTC 
Hi Vinson, can you re-test with ToT?  The scons build is working for me, testing at 134a40d2a67.
Comment 2 Dylan Baker 2017-10-30 17:07:57 UTC 
I think Eric Engstrom sent a fix for this that's been reviewed but hasn't landed yet.
Comment 3 Eric Engestrom 2017-10-30 17:24:14 UTC 
(In reply to Dylan Baker from comment #2)
> I think Eric Engstrom sent a fix for this that's been reviewed but hasn't
> landed yet.

Indeed, I just pushed it:

commit 2117d0331020874c28bb66b0596467f960259eb7
Author: Eric Engestrom <eric@engestrom.ch>
Date:   Sun Oct 29 22:06:28 2017 +0000

    git_sha1_gen: create empty file in fallback path
    
    I missed this part in my conversion, the old stream redirection meant
    the file was always created.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103496
    Fixes: 7088622e5fb506b64c90 "buildsys: move file regeneration logic to
           the script itself"
    Signed-off-by: Eric Engestrom <eric@engestrom.ch>
    Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Comment 4 Ian Romanick 2017-10-30 22:25:50 UTC 
I was just about to report the same bug... but 2117d033 seems to fix it for me.  I'm going to close it.  Vinson should reopen this bug if it's not fixed for him.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.