Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 566244

Summary: Need to add a new platform "aarch64" so simrel
Product: [Technology] CBI Reporter: Sravan Kumar Lakkimsetti <sravankumarl>
Component: CBI p2 Repository AggregatorAssignee: CBI Inbox <cbi-inbox>
Status: CLOSED FIXED QA Contact: David Williams <david_williams>
Severity: normal    
Priority: P3 CC: akurtakov, frederic.gurr, ilg, mistria
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 10   
Whiteboard:
Bug Depends on:    
Bug Blocks: 565419    
Attachments:
Description Flags
Architecture list none

Description Sravan Kumar Lakkimsetti CLA 2020-08-21 01:46:59 EDT
aarch64 is not available in the dropdown of configuration. Can you please let me know how to add this?
Comment 1 Frederic Gurr CLA 2020-08-21 04:59:44 EDT
Which dropdown are you referring to?
Comment 2 Sravan Kumar Lakkimsetti CLA 2020-08-21 05:22:22 EDT
Created attachment 283927 [details]
Architecture list

I would like add a new configuration as gtk, linux with architecture aarch64. If see the screen shot, aarch is not listed. 

My question is how to add aarch64 to the configuration
Comment 3 Mickael Istria CLA 2020-08-21 05:35:04 EDT
Have you tried manually editing the arrg file? Maybe the aggregator model is more tolerant than the tool and wouldn't mind receiving aarch64, so you could make some progress without need to fix the tool as a pre-requisite.
Comment 4 Sravan Kumar Lakkimsetti CLA 2020-08-21 05:38:51 EDT
(In reply to Mickael Istria from comment #3)
> Have you tried manually editing the arrg file? Maybe the aggregator model is
> more tolerant than the tool and wouldn't mind receiving aarch64, so you
> could make some progress without need to fix the tool as a pre-requisite.

I tried that but it had thrown me an error as unrecognized architecture
Comment 5 Frederic Gurr CLA 2020-08-21 05:43:30 EDT
I'll take a look.
Comment 6 Mickael Istria CLA 2020-08-21 05:49:30 EDT
(In reply to Sravan Kumar Lakkimsetti from comment #4)
> I tried that but it had thrown me an error as unrecognized architecture

Ok, too bad. That's what happens when a metamodel is designed assuming it will be forever maintained and ten duplicates constant definitions for something it doesn't control instead of inferring them from something more meaningful...
You may need to fix https://git.eclipse.org/c/cbi/org.eclipse.cbi.p2repo.aggregator.git/tree/org.eclipse.cbi.p2repo.aggregator/model/Aggregator.ecore#n340 , then regenerate the genmodel and the generated Java code. And/or you can also modify directly https://git.eclipse.org/c/cbi/org.eclipse.cbi.p2repo.aggregator.git/tree/org.eclipse.cbi.p2repo.aggregator/src/org/eclipse/cbi/p2repo/aggregator/Architecture.java . I don't know what's best for this aggregator.
Good luck!
PS: I believe in the meantime, Tycho has supported aarch64 without extra pain and investment, maybe it's time to just realize that keeping SimRel and p2 aggregator is  not really profitable and move to greener fields only, like EPP and Tycho...
Comment 7 Sravan Kumar Lakkimsetti CLA 2020-08-25 02:31:57 EDT
(In reply to Frederic Gurr from comment #5)
> I'll take a look.

please add arm64 and aarch64 to the architecture list. arm64 is the keyword used by Mac for apple silicon and aarch64 is the keyword used by Raspberry Pi.
Comment 8 Liviu Ionescu CLA 2020-08-25 04:25:18 EDT
> arm64 is the keyword used by Mac for apple silicon

Is this the value returned by `uname -m` on new Macs?
Comment 9 Sravan Kumar Lakkimsetti CLA 2020-08-25 04:30:58 EDT
(In reply to Liviu Ionescu from comment #8)
> > arm64 is the keyword used by Mac for apple silicon
> 
> Is this the value returned by `uname -m` on new Macs?

We don't have new macs yet. but from the documentation that is the architecture string
Comment 10 Frederic Gurr CLA 2020-08-25 08:16:52 EDT
I've added both architectures. Please install and test it:

=> https://download.eclipse.org/cbi/updates/aggregator/ide/4.13/I20200825-1209/
Comment 11 Sravan Kumar Lakkimsetti CLA 2020-08-25 09:02:22 EDT
(In reply to Frederic Gurr from comment #10)
> I've added both architectures. Please install and test it:
> 
> =>
> https://download.eclipse.org/cbi/updates/aggregator/ide/4.13/I20200825-1209/

It is working. Can you please add I20200825-1209 to https://download.eclipse.org/cbi/updates/aggregator/headless/4.13/ repo? 

Gerrit is failing because of that https://ci.eclipse.org/simrel/job/simrel.runaggregator.VALIDATE.gerrit/1640/console
Comment 12 Frederic Gurr CLA 2020-08-25 09:42:53 EDT
(In reply to Sravan Kumar Lakkimsetti from comment #11)
> It is working. Can you please add I20200825-1209 to
> https://download.eclipse.org/cbi/updates/aggregator/headless/4.13/ repo? 
Done.
Comment 13 Sravan Kumar Lakkimsetti CLA 2020-08-25 09:54:49 EDT
Its working now. You can close this bug. Thanks for quick response
Comment 14 Frederic Gurr CLA 2020-08-25 09:57:18 EDT
Ok. Closing.