diff mbox series

[v1,1/1] package/fb-test-app: Move 'rosetta' prefix from VERSION to SITE

Message ID 20210628130816.29769-1-andriy.shevchenko@linux.intel.com
State Superseded
Headers show
Series [v1,1/1] package/fb-test-app: Move 'rosetta' prefix from VERSION to SITE | expand

Commit Message

Andy Shevchenko June 28, 2021, 1:08 p.m. UTC
Move 'rosetta' prefix from VERSION to SITE to have version be
in a standard format.

Suggested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 package/fb-test-app/fb-test-app.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni June 28, 2021, 1:40 p.m. UTC | #1
On Mon, 28 Jun 2021 16:08:16 +0300
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> Move 'rosetta' prefix from VERSION to SITE to have version be
> in a standard format.
> 
> Suggested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Thanks for the patch. However, are you sure you tested it? I believe
this doesn't build, as the .hash file needs to be adjusted accordingly.

Best regards,

Thomas
Andy Shevchenko June 28, 2021, 3:59 p.m. UTC | #2
On Mon, Jun 28, 2021 at 03:40:08PM +0200, Thomas Petazzoni wrote:
> On Mon, 28 Jun 2021 16:08:16 +0300
> Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> 
> > Move 'rosetta' prefix from VERSION to SITE to have version be
> > in a standard format.
> > 
> > Suggested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> Thanks for the patch. However, are you sure you tested it? I believe
> this doesn't build, as the .hash file needs to be adjusted accordingly.

You are right, it wasn't tested on the repo which has this package enabled in
the configuration.
I'll send v2 which I have just tested:

--2021-06-28 18:58:28--  https://github.com/andy-shev/fb-test-app/archive/rosetta-1.1.1/fb-test-app-1.1.1.tar.gz
...
fb-test-app-1.1.1.tar.gz: OK (sha256: 45d490ed78a6e4425d9a760e81e99dc503af01704e17ab5bf186b87a31c5e3db)
diff mbox series

Patch

diff --git a/package/fb-test-app/fb-test-app.mk b/package/fb-test-app/fb-test-app.mk
index f6e949015ef8..89d533ab15b4 100644
--- a/package/fb-test-app/fb-test-app.mk
+++ b/package/fb-test-app/fb-test-app.mk
@@ -4,8 +4,8 @@ 
 #
 ################################################################################
 
-FB_TEST_APP_VERSION = rosetta-1.1.1
-FB_TEST_APP_SITE = $(call github,andy-shev,fb-test-app,$(FB_TEST_APP_VERSION))
+FB_TEST_APP_VERSION = 1.1.1
+FB_TEST_APP_SITE = $(call github,andy-shev,fb-test-app,rosetta-$(FB_TEST_APP_VERSION))
 FB_TEST_APP_LICENSE = GPL-2.0
 FB_TEST_APP_LICENSE_FILES = COPYING