Chitlesh GOORAH | 10 May 21:27 2009

Re: [Fedora-electronic-lab] I need guidance

On Sun, May 10, 2009 at 4:27 PM, Aveek Sen
<aveeksen@...> wrote:
> Hi!,
>
> I am a first year student of Electronics & Communication Engineering
> at NIT Agartala, India. I am the Fedora ambassador of that region.
>
> I want to know what I need to learn so that I am competent enough to
> make some additions to the FEL, in the form of something related to
> electronics.

Hello Aveek,

I'm looking for a command-wise compatibility of Alliance with Design
Compiler. Alliance is used for Digital design and Design Compiler is
an industry EDA tool.

If "herb" developers make some progress in the future, I will replace
alliance by herb. Herb is a fork of alliance.

The problem with Alliance/herb is that it demands a hard learning
curve. Thereby many people get fed up with alliance. At the same time,
a lot of the manual tasks(things which should be automated) are
required by the designer.

A simple example, to report the timing of any path, it takes a lot of
effort to first find the path. Hence as a designer point of view, I'm
spending 80% of the design time on the software rather  on the design
itself. This is very bad.

(Continue reading)

Gilaras Drakeson | 25 Mar 16:31 2009
Picon

[PATCH] abl: minor - fix function argument types.

Add the type of the arguments of functions that are being passed on to
other functions as arguments. The types are found by looking at the
functions' implementations.
---
 src/abl/ablget.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/abl/ablget.c b/src/abl/ablget.c
index 05ca598..ddc9af3 100644
--- a/src/abl/ablget.c
+++ b/src/abl/ablget.c
 <at>  <at>  -444,7 +444,7  <at>  <at>  long getablexprnumocc (ablexpr *Expr, char *Name)
 |                                                             |
 \------------------------------------------------------------*/

-ablexpr *getablexprmax (int (*Function) (/* ??? */), ablexpr *Expr)
+ablexpr *getablexprmax (int (*Function) (ablexpr *), ablexpr *Expr)
 {
         ablexpr *MaxExpr;
         long Maximum;
 <at>  <at>  -476,7 +476,7  <at>  <at>  ablexpr *getablexprmax (int (*Function) (/* ??? */), ablexpr *Expr)
 |                                                             |
 \------------------------------------------------------------*/

-ablexpr *getablexprmin (int (*Function) (/* ??? */), ablexpr *Expr)
+ablexpr *getablexprmin (int (*Function) (ablexpr *), ablexpr *Expr)
 {
         ablexpr *MinExpr;
         long Minimum;
--

-- 
(Continue reading)

Mukund Sivaraman | 25 Mar 10:51 2009

Copyright discussion with Alliance authors (contd..)

Hi Jean-Paul

On Wed, Mar 25, 2009 at 10:25:48AM +0100, Jean-Paul Chaput wrote:
> I strongly backup the position of Ludovic Jacomme.
> You must restore author's names, license GPLv2 and banners.

This is what the commit log for the changes looked like when made in
Feb:

https://www.banu.com/git/?p=herb.git;a=commitdiff;h=0ad92f2c76ad02c16d3c18f7e4563f5ca8faeacd

The copyright notice and names of authors are preserved. The author
names have been moved to the AUTHORS file, and this is not illegal. We
didn't want large banners (for authors, functions, structures,
definitions, etc.) in the source code files as they add bulk, and we
are removing them slowly. This is our project's preference.

There is no intention of malice on our part. We are merely redoing the
codebase to how we want it, and this is permitted and not illegal under
the license conditions.

		Mukund
Mukund Sivaraman | 25 Mar 10:41 2009

Re: Copyright notice discussion (with Alliance team)

Hi Jean-Paul

On Wed, Mar 25, 2009 at 10:25:48AM +0100, Jean-Paul Chaput wrote:
> I strongly backup the position of Ludovic Jacomme.
> You must restore author's names, license GPLv2 and banners.

These were not removed as far as I know:

1. All the copyright notices which were present before should be there.

2. Author names are moved to AUTHORS file which is where it belongs in
a GNU style project.

3. GPL v3 license upgrade is entirely permitted.

		Mukund
Mukund Sivaraman | 25 Mar 10:38 2009

License change discussion (with Alliance team)

On Wed, Mar 25, 2009 at 09:58:09AM +0100, Ludovic Jacomme wrote:
> Worst of all, you change the license GPL V2 to V3 and you forgot to
> let alliance libraries in LGPL ! Who are you to change the license
> term of Alliance CAD tools ?

This is perfectly allowed under the terms of the GNU GPL and GNU LGPL
licenses under which Alliance is distributed. Please go through the
license texts here:

http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html

Specifically, because Alliance is distributed under "either version 2 of
the License, or (at your option) any later version", we are able to
use GNU GPLv3 with it.

Similarly, upgrading from LGPL to GPL is also allowed. It is considered
an upgrade by the FSF as GPL protects freedoms more :) We did this to
keep things more open than closed.

Also, the decision to change the license for libraries were made after
discussing two points:

1. We don't think many people are going to make tools which use Herb
libraries and actually re-distribute these. Some may make site-specific
tools, but this can be done with GNU GPL too.

2. Moving code around from tools to libraries and vice versa would
involve license considerations which we didn't want to deal with. We
plan to put functionality present in tools into libraries, so we can
(Continue reading)

Mukund Sivaraman | 25 Mar 10:31 2009

Herb copyrights discussion

Hi Ludovic

This is a pretty serious accusation. I have replied to your points
below.

On Wed, Mar 25, 2009 at 08:34:49AM +0100, Ludovic Jacomme wrote:
> Having a look in herb sources, you have modified all headers in .c
> file removing author names, credits, copyright stuff and put Herb
> project name instead of LIP6 one  !!! You have remove banners in
> graphical tools ...
> 

We have not removed copyright notices as far as I know. Every file
should contain the LIP6 copyright notice, which is what Alliance is
distributed under. Only the name of the project has been changed to
Herb everywhere. This is to make clear that Herb is a different project
from Alliance. It follows the banner suggested in the GNU GPL license
itself.

We can't claim our project is Alliance, because it would be trademark
infringement.. and the project is certainly not Alliance. It is a fork.
Similar to how Inkscape is a fork of Sodipodi, etc. So names have to be
changed.

> Shame on you and btw it's a copyright infringement ! You have removed
> my name in more than 1700 source code files !

How is moving a name to the AUTHORS file copyright infringement? No
copyright notices have been removed.

(Continue reading)

Gilaras Drakeson | 24 Mar 17:39 2009
Picon

[PATCH 1/4] abe: Fix missing prototypes warnings.

Some of these files were processed using protoize (part of gcc).
for instance:
    protoize -c "$INCLUDES" some-file.c
where INCLUDES where extracted from the Makefile.

Also a minor bug in beh_view.c (in the body of beh_viewbeaux) is
fixed.  As a side effect, two previously hidden bugs in
src/flatbeh/utils.c were also caught and fixed. They were cases of
passing wrong number of arguments to functions. These were caught when
the prototypes were fixed.

Add a few #includes ("mut.h", "aut.h", "abl.h", "abe.h") in 3 of the
files, since they were not including "abe.h".
---
 src/abe/abe.h          |  156 ++++++++++++++++++++++++------------------------
 src/abe/beh_add.c      |  129 ++++++++++++++++++++--------------------
 src/abe/beh_del.c      |   92 ++++++++++++++--------------
 src/abe/beh_dict.c     |   36 ++---------
 src/abe/beh_error.c    |   10 ++--
 src/abe/beh_fre.c      |   52 ++++++++--------
 src/abe/beh_getgenva.c |    6 +-
 src/abe/beh_message.c  |   10 ++--
 src/abe/beh_rmv.c      |   76 ++++++++++++------------
 src/abe/beh_toolbug.c  |   10 +--
 src/abe/beh_view.c     |   42 ++++----------
 src/abe/main.c         |    7 +--
 src/flatbeh/utils.c    |    6 +-
 13 files changed, 291 insertions(+), 341 deletions(-)

diff --git a/src/abe/abe.h b/src/abe/abe.h
(Continue reading)

Gilaras Drakeson | 24 Mar 12:15 2009
Picon

[PATCH 1/2] flatbeh: Fix indentation

---
 src/flatbeh/abstract.c |  518 ++++++++++++++++++++++++------------------------
 src/flatbeh/main.c     |   16 +-
 src/flatbeh/utils.c    |   70 ++++----
 src/flatbeh/utils.h    |    8 +-
 4 files changed, 306 insertions(+), 306 deletions(-)

diff --git a/src/flatbeh/abstract.c b/src/flatbeh/abstract.c
index 28d55a6..b14ec93 100644
--- a/src/flatbeh/abstract.c
+++ b/src/flatbeh/abstract.c
 <at>  <at>  -28,13 +28,13  <at>  <at> 
 #include "utils.h"
 
 /*------------------------------------------------------------------------------
-create	 	: Regenere un ABL en substituant les variables formelles
-                  par les signaux de la lofig
---------------------------------------------------------------------------------
-parametres  	: en ABL, un modele d'instance et une table de hasch.
---------------------------------------------------------------------------------
-return		: un ABL.
-------------------------------------------------------------------------------*/
+  create	 	: Regenere un ABL en substituant les variables formelles
+  par les signaux de la lofig
+  --------------------------------------------------------------------------------
+  parametres  	: en ABL, un modele d'instance et une table de hasch.
+  --------------------------------------------------------------------------------
+  return		: un ABL.
+  ------------------------------------------------------------------------------*/
 chain_list *create (expr, model, tableTH)
(Continue reading)

Gilaras Drakeson | 24 Mar 12:15 2009
Picon

[PATCH 1/2] abe: Fix indentation

Fix indentation and also replace
  #ifdef __cplusplus
  extern "C" {
  #endif                          /* __cplusplus */
with
  G_BEGIN_DECLS
---
 src/abe/abe.h      |  503 ++++++++++++++++++++++++++--------------------------
 src/abe/beh_dict.c |    6 +-
 src/abe/main.c     |  110 ++++++------
 3 files changed, 308 insertions(+), 311 deletions(-)

diff --git a/src/abe/abe.h b/src/abe/abe.h
index 4129d57..d099de6 100644
--- a/src/abe/abe.h
+++ b/src/abe/abe.h
 <at>  <at>  -20,262 +20,259  <at>  <at> 
 #ifndef ABE_BEHDEF
 #define ABE_BEHDEF

-#ifdef __cplusplus
-extern "C" {
-#endif                          /* __cplusplus */
+G_BEGIN_DECLS

-        /* ###------------------------------------------------------### */
-        /*    defines                                                   */
-        /* ###------------------------------------------------------### */
+/* ###------------------------------------------------------### */
+/*    defines                                                   */
(Continue reading)

Gilaras Drakeson | 23 Mar 19:03 2009
Picon

[PATCH] fixing missing prototype warnings in src/abe.

Some of these files were processed using protoize (part of gcc).
for instance:
    protoize -c "$INCLUDES" some-file.c
where INCLUDES where extracted from the Makefile.

also a minor bug in beh_view.c (in the body of beh_viewbeaux) is fixed.
---
 src/abe/abe.h          |  679 ++++++++++++++++++++++++++++++------------------
 src/abe/beh_add.c      |  129 +++++-----
 src/abe/beh_del.c      |   92 ++++----
 src/abe/beh_dict.c     |   36 +---
 src/abe/beh_error.c    |    9 +-
 src/abe/beh_fre.c      |   52 ++--
 src/abe/beh_getgenva.c |    6 +-
 src/abe/beh_message.c  |   11 +-
 src/abe/beh_rmv.c      |   76 +++---
 src/abe/beh_toolbug.c  |   10 +-
 src/abe/beh_view.c     |   42 +---
 11 files changed, 632 insertions(+), 510 deletions(-)

diff --git a/src/abe/abe.h b/src/abe/abe.h
index 4129d57..4c7c07e 100644
--- a/src/abe/abe.h
+++ b/src/abe/abe.h
 <at>  <at>  -20,262 +20,435  <at>  <at> 
 #ifndef ABE_BEHDEF
 #define ABE_BEHDEF
 
-#ifdef __cplusplus
-extern "C" {
(Continue reading)

Gilaras Drakeson | 23 Mar 19:14 2009
Picon

[PATCH 2/2] minor - fix indentation

---
 src/abe/beh_add.c      |  102 ++++++++++++++++++++++++------------------------
 src/abe/beh_del.c      |   68 ++++++++++++++++----------------
 src/abe/beh_dict.c     |    6 +-
 src/abe/beh_fre.c      |   26 ++++++------
 src/abe/beh_getgenva.c |    4 +-
 src/abe/beh_rmv.c      |   56 +++++++++++++-------------
 6 files changed, 131 insertions(+), 131 deletions(-)

diff --git a/src/abe/beh_add.c b/src/abe/beh_add.c
index 5f5db00..cf5e760 100644
--- a/src/abe/beh_add.c
+++ b/src/abe/beh_add.c
 <at>  <at>  -29,10 +29,10  <at>  <at> 

 struct beaux *beh_addbeaux (struct beaux *lastbeaux, char *name, struct chain *abl_expr, struct
bddnode *bdd_expr)

-                                /* pointer on the last beaux structure  */
-                                /* signal's name                        */
-                                /* signal's expression (ABL)            */
-                                /* signal's expression (BDD)            */
+/* pointer on the last beaux structure  */
+/* signal's name                        */
+/* signal's expression (ABL)            */
+/* signal's expression (BDD)            */

 {
         struct beaux *ptaux;
 <at>  <at>  -66,11 +66,11  <at>  <at>  struct beaux *beh_addbeaux (struct beaux *lastbeaux, char *name, struct chain *a
(Continue reading)


Gmane