[dpdk-dev] [PATCH] build system: apps will use

2016-08-06 Thread Burt Silverman
To bring the coding style for applications like VPP into line with standard
practices, one expects them to include system header files with the path
starting from /usr/include, some examples:

#include 
#include 
#include 
and thus
#include 

Up until now, this has not been possible, and instead, we see things like
#include 

The goal is to give a professional, open source, feel to projects like VPP
and DPDK.

Burt Silverman (1):
  build system: apps will use 

 app/cmdline_test/cmdline_test.c|  8 +--
 app/cmdline_test/commands.c| 10 ++--
 app/pdump/main.c   | 26 -
 app/proc_info/main.c   | 32 +--
 app/test-acl/main.c| 10 ++--
 app/test-pipeline/config.c | 58 +--
 app/test-pipeline/init.c   | 58 +--
 app/test-pipeline/main.c   | 56 +-
 app/test-pipeline/pipeline_acl.c   | 18 +++---
 app/test-pipeline/pipeline_hash.c  | 18 +++---
 app/test-pipeline/pipeline_lpm.c   | 18 +++---
 app/test-pipeline/pipeline_lpm_ipv6.c  | 18 +++---
 app/test-pipeline/pipeline_stub.c  |  8 +--
 app/test-pipeline/runtime.c| 58 +--
 app/test-pmd/cmdline.c | 66 +++---
 app/test-pmd/config.c  | 44 +++
 app/test-pmd/csumonly.c| 58 +--
 app/test-pmd/flowgen.c | 52 -
 app/test-pmd/icmpecho.c| 38 ++---
 app/test-pmd/ieee1588fwd.c |  4 +-
 app/test-pmd/iofwd.c   | 46 +++
 app/test-pmd/macfwd.c  | 48 
 app/test-pmd/macswap.c | 48 
 app/test-pmd/parameters.c  | 46 +++
 app/test-pmd/rxonly.c  | 50 
 app/test-pmd/testpmd.c | 54 +-
 app/test-pmd/txonly.c  | 54 +-
 app/test/commands.c| 48 
 app/test/packet_burst_generator.c  |  4 +-
 app/test/packet_burst_generator.h  | 10 ++--
 app/test/resource.c|  2 +-
 app/test/resource.h|  4 +-
 app/test/test.c| 22 
 app/test/test.h|  8 +--
 app/test/test_acl.c| 12 ++--
 app/test/test_alarm.c  | 12 ++--
 app/test/test_atomic.c | 16 +++---
 app/test/test_byteorder.c  |  2 +-
 app/test/test_cmdline_cirbuf.c |  4 +-
 app/test/test_cmdline_etheraddr.c  |  8 +--
 app/test/test_cmdline_ipaddr.c |  6 +-
 app/test/test_cmdline_lib.c| 10 ++--
 app/test/test_cmdline_num.c|  6 +-
 app/test/test_cmdline_portlist.c   |  4 +-
 app/test/test_cmdline_string.c |  8 +--
 app/test/test_common.c |  4 +-
 app/test/test_cpuflags.c   |  4 +-
 app/test/test_cryptodev.c  | 18 +++---
 app/test/test_cryptodev_aes.c  | 18 +++---
 app/test/test_cryptodev_perf.c | 16 +++---
 app/test/test_cycles.c |  4 +-
 app/test/test_debug.c  |  6 +-
 app/test/test_devargs.c|  4 +-
 app/test/test_distributor.c| 10 ++--
 app/test/test_distributor_perf.c   | 10 ++--
 app/test/test_eal_flags.c  |  4 +-
 app/test/test_errno.c  |  6 +-
 app/test/test_func_reentrancy.c| 42 +++---
 app/test/test_hash.c   | 26 -
 app/test/test_hash_functions.c | 10 ++--
 app/test/test_hash_multiwriter.c   | 14 ++---
 app/test/test_hash_perf.c  | 18 +++---
 app/test/test_hash_scaling.c   | 10 ++--
 app/test/test_interrupts.c |  6 +-
 app/test/test_ivshmem.c|  2 +-
 app/test/test_kni.c| 10 ++--
 app/test/test_kvargs.c |  4 +-
 app/test/test_link_bonding.c   | 20 +++
 app/test/test_link_bonding_mode4.c | 26 -
 app/test

[dpdk-dev] test: failed to clone dpdk source in windows

2016-08-08 Thread Burt Silverman
Oddly enough, it seems to work fine under cygwin. That should solve your
issue.

On Sun, Aug 7, 2016 at 11:50 PM, linhaifeng  wrote:

> ? 2016/8/7 4:33, Jan Viktorin ??:
> > On Fri, 05 Aug 2016 09:51:06 +0200
> > Thomas Monjalon  wrote:
> >
> >> 2016-08-05 09:44, Thomas Monjalon:
> >>> 2016-08-05 10:09, linhaifeng:
>  hi,thomas
> 
>  Could you change the name of file in directory
> app/test/test_pci_sysfs/bus/pci/devices/ ?
>  I think somebody like us also cann't access internet in
> liunux.Windows not support file name
>  include ':'.
> >
> > Hello,
> >
> > I am sorry for this situation but (as Thomas mentioned below) Windows is
> > not considered as a supported platform. If you have issues with this
> > approach you are welcome to prepare some patch changing this to do
> > the same thing in another way.
> >
> > There was a discussion when accepting the fake sysfs into DPDK on the
> > public mailing list. I did not received any serious complaints at that
> > time.
> >
> > Regards
> > Jan
> >
> >>>
> >>> Not sure to understand the use case.
> >>> This is a fake sysfs, we cannot change the names.
> >>
> >> In other words, Windows is not supported, neither for runtime
> >> nor for development.
> >> For information, which filesystem do not support ':'?
> >>
> >> If someone wants to add this support, the fake sysfs could have slightly
> >> different names and rightly renamed during compilation.
> >
>
> I just want to edit code in windows and compile in linux,so i want to
> clone it in windows,but
> it's not support.The message in win7 is :
> $ git clone http://dpdk.org/git/dpdk
> Cloning into 'dpdk'...
> remote: Counting objects: 51912, done.
> remote: Compressing objects: 100% (10331/10331), done.
> rRemote: Total 51912 (delta 41342), reused 51808 (delta 41eceiving
> objects: 100%
>  (51271)912/519   MiB |
> Receiving objects: 100% (51912/51912), 25.99 MiB | 1.99 MiB/s, done.
> Resolving deltas: 100% (41342/41342), done.
> Checking connectivity... done.
> fatal: cannot create directory at 'app/test/test_pci_sysfs/bus/
> pci/devices/:
> 01:00.0': Invalid argument
> warning: Clone succeeded, but checkout failed.
> You can inspect what was checked out with 'git status'
> and retry the checkout with 'git checkout -f HEAD'
>
> If everyone edit codes in linux there is no need to fix it.
>
>
>